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

Gather catnip #86

Closed
oliversalzburg opened this issue Dec 28, 2022 · 1 comment
Closed

Gather catnip #86

oliversalzburg opened this issue Dec 28, 2022 · 1 comment
Assignees
Labels
needs:merge This work has been completed. It needs to be merged prio:important Should be included in next release status:completed This needs no further attention type:feature Completely new behavior
Milestone

Comments

@oliversalzburg
Copy link
Member

Describe the Feature

KS should be able to automatically gather catnip, if you're in a state of the game where this is applicable.

Your Savegame

No response

@oliversalzburg oliversalzburg added type:feature Completely new behavior prio:beta needs:triage A developer needs to look at this labels Dec 28, 2022
@oliversalzburg oliversalzburg self-assigned this Dec 28, 2022
@oliversalzburg oliversalzburg added good first issue If you haven't contributed yet, this might be a good place to start needs:work It's clear what needs to be done. Now we need to do it and removed needs:triage A developer needs to look at this labels Dec 28, 2022
@oliversalzburg oliversalzburg added this to the v2.0-rc milestone Dec 28, 2022
@WrayOfSunshine
Copy link

FWIW Script Kitties handled this with the following function, which was then called as part of a core "check everything to see what we need to do next" loop:

function autoNip() {
	if (gamePage.bld.buildingsData[0].val < 30) {
		$(".btnContent:contains('Gather')").trigger("click");

		// Set the triggerImmediate flag for this function, so that it is called again next tick
		dispatchFunctions.autoNip.triggerImmediate = true;
	}
}

Obviously the implementation in kitten-scientists will vary somewhat, but hopefully that helps for anyone who is taking this on.

@oliversalzburg oliversalzburg added prio:important Should be included in next release needs:merge This work has been completed. It needs to be merged status:completed This needs no further attention and removed good first issue If you haven't contributed yet, this might be a good place to start needs:work It's clear what needs to be done. Now we need to do it labels Dec 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs:merge This work has been completed. It needs to be merged prio:important Should be included in next release status:completed This needs no further attention type:feature Completely new behavior
Projects
Development

No branches or pull requests

2 participants