Skip to content

Api transaction count fix#12118

Merged
ThatSpaceGuy merged 7 commits intomainfrom
api_transaction_count_fix
Apr 30, 2025
Merged

Api transaction count fix#12118
ThatSpaceGuy merged 7 commits intomainfrom
api_transaction_count_fix

Conversation

@koseni123
Copy link
Contributor

🎫 Ticket

Link to the relevant ticket:
LG-67

🛠 Summary of changes

  • Change date range to use time format as opposed to date format which caused an error
  • Removed title from emailable report function

📜 Testing Plan

Provide a checklist of steps to confirm the changes.

  • Go to API Transaction count report
  • Time range should have the to_time method.
  • Emailable report should no longer have the title, just a header

Copy link
Contributor

@ThatSpaceGuy ThatSpaceGuy left a comment

Choose a reason for hiding this comment

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

Looks good - thank you!

@ThatSpaceGuy ThatSpaceGuy merged commit 64fd725 into main Apr 30, 2025
1 check passed
@ThatSpaceGuy ThatSpaceGuy deleted the api_transaction_count_fix branch April 30, 2025 14:57
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