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

Add a CLI reporting tool #9972

Merged
merged 7 commits into from
Jan 3, 2025
Merged

Add a CLI reporting tool #9972

merged 7 commits into from
Jan 3, 2025

Conversation

steven-sheehy
Copy link
Member

@steven-sheehy steven-sheehy commented Dec 16, 2024

Description:

  • Add a reporting tool that produces a CSV file with the list of transactions and running balance for the given accounts
  • Add a note on how to smoke test the GraphQL module
  • Limit how much data Tomcat will read from a POST request
  • Remove an unused variable in the monitor after a recent refactor

Related issue(s):

Fixes #9918

Notes for reviewer:

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

Signed-off-by: Steven Sheehy <[email protected]>
@steven-sheehy steven-sheehy added enhancement Type: New feature javascript Pull requests that update Javascript code labels Dec 16, 2024
@steven-sheehy steven-sheehy added this to the 0.121.0 milestone Dec 16, 2024
@steven-sheehy steven-sheehy self-assigned this Dec 16, 2024
Signed-off-by: Steven Sheehy <[email protected]>
Signed-off-by: Steven Sheehy <[email protected]>
Copy link

codecov bot commented Dec 17, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.33%. Comparing base (da000d7) to head (050d979).
Report is 46 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #9972      +/-   ##
============================================
+ Coverage     92.21%   92.33%   +0.11%     
+ Complexity     7840     7753      -87     
============================================
  Files           954      938      -16     
  Lines         32791    32663     -128     
  Branches       4141     4131      -10     
============================================
- Hits          30239    30159      -80     
+ Misses         1581     1533      -48     
  Partials        971      971              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Signed-off-by: Steven Sheehy <[email protected]>
Signed-off-by: Steven Sheehy <[email protected]>
@steven-sheehy steven-sheehy marked this pull request as ready for review January 3, 2025 01:27
@steven-sheehy steven-sheehy requested review from a team as code owners January 3, 2025 01:27
Signed-off-by: Steven Sheehy <[email protected]>
Copy link

sonarqubecloud bot commented Jan 3, 2025

Copy link
Contributor

@jnels124 jnels124 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@xin-hedera xin-hedera left a comment

Choose a reason for hiding this comment

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

LGTM

@steven-sheehy steven-sheehy merged commit f87bed8 into main Jan 3, 2025
35 of 36 checks passed
@steven-sheehy steven-sheehy deleted the 9918-report-tool branch January 3, 2025 19:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Type: New feature javascript Pull requests that update Javascript code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Generate an account report in CSV format
3 participants