Skip to content
This repository was archived by the owner on May 20, 2025. It is now read-only.

feat(analytics): separate base token activity analytic into booked and transferred counts #960

Merged
merged 19 commits into from
Dec 12, 2022

Conversation

Alex6323
Copy link
Collaborator

@Alex6323 Alex6323 commented Dec 5, 2022

Linked Issues

Notes to Reviewer

As a reviewer, please pay particular attention to the following areas when reviewing this PR and tick the above boxes after you have completed the steps.

API Changes

  • Test the API endpoints which were added/changed.

Database Changes

  • Review database queries for correctness/conciseness.
  • Ensure queries are supported by indexes if needed.
  • Check for breaking changes in the data model and matching (conventional) commit message prefix.

@Alex6323 Alex6323 force-pushed the fix/db/base-token-activity-query branch from 807fedc to 2954edf Compare December 6, 2022 10:45
@Alex6323 Alex6323 force-pushed the fix/db/base-token-activity-query branch from 2030263 to 30deb1f Compare December 8, 2022 22:28
@Alex6323 Alex6323 changed the title fix(db): fix base token activity query feat(db): separate base token activity analytic into booked and transferred counts Dec 12, 2022
@Alex6323 Alex6323 changed the title feat(db): separate base token activity analytic into booked and transferred counts feat(analytics): separate base token activity analytic into booked and transferred counts Dec 12, 2022
@grtlr grtlr merged commit 5a533bd into main Dec 12, 2022
@grtlr grtlr deleted the fix/db/base-token-activity-query branch December 12, 2022 14:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Token transferred analytics should be transaction-based
2 participants