SyfSchydea / pokeclicker-automation

Automates various aspects of the game Pokéclicker
1 stars 0 forks source link

AutoQuest uses wrong quest collection method #12

Closed SyfSchydea closed 1 year ago

SyfSchydea commented 1 year ago

Currently, it uses App.game.quests.currentQuests()[questIdx].claim(), but it should be using App.game.quests.claimQuest(questIdx).

The current method does not check if all quests are complete, which can lead to being left with a full list of completed quests, which doesn't normally happen.

SyfSchydea commented 1 year ago

Note that claimQuest expects an index to App.game.quests.questList(), whereas, we were previously using indices to App.game.quests.currentQuests()

SyfSchydea commented 1 year ago

It seems we can convert by fetching App.game.quests.currentQuests()[currentQuestIdx].index