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

Last harvest for Xaquelina or Elle quest does not show up in counters #189

Open
NetSysFire opened this issue May 1, 2022 · 1 comment
Open

Comments

@NetSysFire
Copy link

NetSysFire commented May 1, 2022

Steps to reproduce:

  1. Get your daily quest from Xaquelina or do an Elle one, I recommend the former.
  2. Make sure the session counter for that specific thing you are going to harvest is 0 (so it is easier to spot).
  3. She tells you to e.g get 60 Red Snapdragons from WS.
  4. Harvest them.
  5. After you get the popup that you got all of them, check your counters. It will be less than 60.

Expected behavior: The last harvest gets registered in the counters, too.

Spoke with radu about this yesterday, he said I should report it because it might be a client bug. So here it is.

Edit: I do not know how #ach behaves in this case. I am not sure if it registers it or not. I can not test because I harvested over 65001 plants.

@pjbroad
Copy link
Collaborator

pjbroad commented May 3, 2022

I can confirm the issue. Looking at the message sequence, it looks to be a server issue. Here's some debug I added:

got new inventory item quantity=54 image_id=27
got new inventory item quantity=56 image_id=27
got new inventory item quantity=58 image_id=27
stopping harvest [You stopped harvesting.]
got new inventory item quantity=60 image_id=27

The "You stopped harvesting" messages, due to you completing the quest, comes before the last items arrive in the inventory. So the client does not recognise the new items were harvested.

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

No branches or pull requests

2 participants