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

PP-10608: Correcting the number of days in transaction queries. #4178

Merged
merged 1 commit into from
Feb 23, 2024

Conversation

nataliecarey
Copy link
Contributor

WHAT

As described in PP-10608 the number of days searched on the dashboard is currently out by one. It was intentionally written (with comments describing it) but having looked deeper into it because of the use of midnight as the start of the period the extra day wasn't needed

HOW

On an account with transactions every day visit the dashboard and select "Previous 7 days" or "Previous 30 days", the query was previously showing a date range one day longer than requested for each of these categories.

@nlsteers
Copy link
Collaborator

Code change looks fine, there's a small accordion on the dashboard service view that explains how the values are calculated. Is this still correct or does it need updating in line with the changes in this PR?

See example image:
Screenshot 2024-02-22 at 16 44 11

@nataliecarey
Copy link
Contributor Author

Code change looks fine, there's a small accordion on the dashboard service view that explains how the values are calculated. Is this still correct or does it need updating in line with the changes in this PR?

See example image: Screenshot 2024-02-22 at 16 44 11

Good spot. I'll check, we should be updating it and I expect it's updated as a side-effect of my change but I'll confirm.

@nataliecarey
Copy link
Contributor Author

Code change looks fine, there's a small accordion on the dashboard service view that explains how the values are calculated. Is this still correct or does it need updating in line with the changes in this PR?

See example image: Screenshot 2024-02-22 at 16 44 11

I've tested, it shows correctly in the accordion 🎉

Copy link
Collaborator

@nlsteers nlsteers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@nataliecarey nataliecarey changed the title PP-10608: Correcting the number of daysin transaction queries. PP-10608: Correcting the number of days in transaction queries. Feb 23, 2024
@nataliecarey nataliecarey merged commit 29b8497 into master Feb 23, 2024
12 checks passed
@nataliecarey nataliecarey deleted the PP-10608_correcting_days_of_transactions branch February 23, 2024 13:44
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.

2 participants