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

fix(Stats): fix User & Location stats after allowing any user to add prices on PRICE_TAG proofs #668

Merged
merged 4 commits into from
Jan 2, 2025

Conversation

raphodn
Copy link
Member

@raphodn raphodn commented Jan 2, 2025

What

Following #609
We need to adapt how some stats are calculated:

  • User location count: do the aggregation on proofs, not on prices
  • Location user count: do the aggregation on proofs, not on prices

Also simplified the User proof count calculation. And update the tests

@raphodn raphodn linked an issue Jan 2, 2025 that may be closed by this pull request
@raphodn raphodn merged commit 2481f14 into main Jan 2, 2025
8 checks passed
@raphodn raphodn deleted the raphodn/stats-count-update branch January 2, 2025 10:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Proofs (Price tags): allow users other than the owner to add prices
1 participant