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

collect pokecoins quest gets insta complete from farm #440

Closed
Claygame opened this issue Sep 17, 2019 · 7 comments · May be fixed by pokeclicker/pokeclicker-dev#163
Closed

collect pokecoins quest gets insta complete from farm #440

Claygame opened this issue Sep 17, 2019 · 7 comments · May be fixed by pokeclicker/pokeclicker-dev#163

Comments

@Claygame
Copy link

i required 46k coins to do the quest but when i farmed a full farm of Citrus berry (which gave me 25k coins) the quest completed instantaneously, i have tested it multiple times but the quest always considers wayyy higher income from farm as compared to the actual amount gained.

@RedSparr0w
Copy link
Member

Hmm, that is a strange bug.
Maybe it's giving double the amount towards quest.
i wonder if you only harvested 1 bush what would happen.

@Claygame
Copy link
Author

i tested some stuff and seems like the pokecoins gain the bonus of Amulet coin key item if it is active when berry are harvested, disabling this should fix it.

@aure2006
Copy link

It's not just amulet coin, but also AchievementHandler.achievementBonus()
https://files.catbox.moe/5vzj8m.txt

@RedSparr0w
Copy link
Member

I might leave this for now, not sure if it's a bad thing that the berries money gains the bonuses 🤔

@RedSparr0w
Copy link
Member

Thoughts on if this is actually an issue, or is it expected that the money gained would be affected by oak items and battle items?

@Claygame
Copy link
Author

I think it can be both good and bad. But the quest seems to give very low qp so it can stay this way i suppose.

@Ishadijcks Ishadijcks transferred this issue from pokeclicker/pokeclicker-dev Dec 29, 2019
@Ishadijcks
Copy link
Collaborator

This seems like intended behaviour, quests check how much money you actually gain, which is after multipliers.

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.

4 participants