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

foraging jobs ignore threshold #150

Open
CodeOptimist opened this issue Oct 10, 2019 · 2 comments
Labels

Comments

@CodeOptimist
Copy link

@CodeOptimist CodeOptimist commented Oct 10, 2019

Describe the bug
selected foraging thing isn't added to count filter (gear menu)

Expected behavior
after selecting it on right side, should show up in filter, like with other tabs

Steps to reproduce

  1. forage tab
  2. click <New job>
  3. select any plant / bush / healroot
  4. check count filter gear menu
  5. observe it empty

Other mods

I wouldn't do you dirty like that. Just HugsLib.

Output log

Save game

Screenshots

Environment
Win10 64

Game
64 Steam 1.0

@dandiez

This comment has been minimized.

Copy link

@dandiez dandiez commented Oct 27, 2019

@CodeOptimist
I could not reproduce the bug... Did you start a new game or was this issue on an existing save?
Do you effectively see an empty dialog (the one with the clear all / allow all buttons below?)
Untitled

@CodeOptimist

This comment has been minimized.

Copy link
Author

@CodeOptimist CodeOptimist commented Oct 27, 2019

@dandiez New games AND existing saves. With only HugsLib and Colony Manager.
Yes, just like that.

If you skip step 2 the counted filter has some defaults, but those also don't update as "Allowed plants" is tweaked.

The Mining tab keeps "Minerals" and the counted filter completely in sync, when I check/uncheck one it updates in the other. I expected Foraging to work like this as well.

Although actually it looks like Foraging allows you to count anything, whereas Mining restricts the count filter to actual minerals.

That's actually cool and I wouldn't mind if all the tabs had independent "do this" and "count this", but it seems selecting a plant to forage should still probably include it to be counted automatically.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.