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

Get monthly average results in statistics filter if the query is less than two years #66

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

jessicaqgomez
Copy link

@jessicaqgomez jessicaqgomez commented Jul 24, 2023

Why this should be merged?

  • The blockchain statistics filter was changed to average the results per month if the query dates are less or equal than two years.

This change was made to limit the number of responses to get a maximum of 30 results by averaging the information within the determined date range per month

How this was tested:

several requests were made to the database and the cache service was executed to verify that the query was successful.
In addition to this, the unit and integration tests were executed to verify that the change had not affected the expected results.

@jessicaqgomez jessicaqgomez requested a review from knikos July 24, 2023 17:15
@jessicaqgomez jessicaqgomez force-pushed the Fix/Blockchain_Statistics branch from 63844e3 to d14cd16 Compare July 24, 2023 17:21
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 this pull request may close these issues.

1 participant