Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Workforce percent dont work #97

Open
mcailleauxgreenskin opened this issue Apr 27, 2023 · 8 comments · Fixed by #99
Open

Workforce percent dont work #97

mcailleauxgreenskin opened this issue Apr 27, 2023 · 8 comments · Fixed by #99

Comments

@mcailleauxgreenskin
Copy link

Hello

Since the last version, the workforce percent dont work anymore, products amounts are not update by changing the workforce percent

image

image

@Stummi
Copy link
Contributor

Stummi commented May 1, 2023

Hey, just want to drop a message that I implemented a Fix for this issue here, which you can see/test on my forks deployment here

@theneofusion
Copy link

Has the change been pushed to the site?
I ask because this still doesn't work properly:
workforce

Note the manually lowered Actual Workforce but the unchanged (maxed) production efficiency and ration/med supplies usage.

@crissian
Copy link
Owner

crissian commented May 2, 2023

I didn’t deploy it yet. I’ll deploy it when I find the time

@crissian
Copy link
Owner

crissian commented May 3, 2023

This doesn't seem to work as expected. I'm sure "it works on your machine"

@crissian crissian reopened this May 3, 2023
@mcailleauxgreenskin
Copy link
Author

Hello,

It's worse, now every time we add a housing module, the percentage is different :'(

image

image

image

@LittlDog
Copy link

LittlDog commented May 6, 2023

Doesn't wanted to open a new topic for it, since this is kinda releated.

Could you add an option to manually add Workforce when adding a Habitat without any other Modules?

@Stummi
Copy link
Contributor

Stummi commented May 7, 2023

Hey, I am really sorry, it seems like my PR introduced a bug with the calculation (I fixed it on my branch, but the PR got already merged in the meantime). I will provide another one to fix this issues

@elidiak
Copy link

elidiak commented May 11, 2023

@Stummi Did confirm that your version works. I'll wait for crissian to merge

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants