Skip to content

feat: Data reports #23

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 75 commits into from
Jan 8, 2025
Merged

feat: Data reports #23

merged 75 commits into from
Jan 8, 2025

Conversation

sophialittlejohn
Copy link
Collaborator

@sophialittlejohn sophialittlejohn commented Dec 2, 2024 •
edited
Loading

Description

This pull request adds all of the remaining data reports:

  • investor transactions
  • asset transactions
  • fee transactions
  • token price
  • asset list
  • investor list

#14

#

@sophialittlejohn
Refactor processing and fix caching and fix tests
474fc24
@sophialittlejohn
Clean up processor and add tests and mocks
c3ab9ad
@sophialittlejohn
Clean up types
89cca6f
@sophialittlejohn
Add cashflow report and tests
562e14c
@sophialittlejohn
Remove defer and use querywith poolid
8625327
@sophialittlejohn
Refactor tests and add comments
710571f
@sophialittlejohn
Add query for metadata and utils around it
0cc968f
@sophialittlejohn
Add data for pool fee snapshots and use in cashflow statments
62a6fff
@sophialittlejohn
Group tranche data by id to prevent double processing
bd50461
@sophialittlejohn
fix zero case in fromFloat wrapper
efc2e37
@sophialittlejohn
Prevent fees and endCashBalance from being aggregated
2580226
@sophialittlejohn
Add fee snapshot mocks and improve fee testing
3c4b0b6
@sophialittlejohn
Add a few more fee test
a059805
@sophialittlejohn
Differentiate public and private credit for cashflows
a205c3d
@sophialittlejohn
Remove identifying info
c4b9997
@sophialittlejohn
Fix data aggregation and improve test coverage with real data
d44e2d8
@sophialittlejohn
Add code coverage support
fd88d5a
@sophialittlejohn
Add coverage summary
2396604
@sophialittlejohn
Exclude types from coverage
452816e
@sophialittlejohn
And P&L report and processor tests
a44d576
@sophialittlejohn
Add tests for applyGrouping
cc8926a
@sophialittlejohn
Add readme for reports
9f1c6b5
@sophialittlejohn
Add test to applyGrouping
22ee830
@sophialittlejohn
Fix test
fabe96e
@sophialittlejohn
Merge branch 'main' of github.com:centrifuge/centrifuge-sdk into reports
df1c10d
@sophialittlejohn
Fix BigInt tests
01879fd
@sophialittlejohn
Clean up metadata fetching
97b6efc
@sophialittlejohn
Remove cache time
db48c58
@sophialittlejohn
Remove filtering in keys
3c672d3
@sophialittlejohn
Fix import so that decimal.js-light works in browser and test env
25fa8bf
gpmayorga
gpmayorga approved these changes Dec 20, 2024
View reviewed changes
Copy link
Collaborator

@gpmayorga gpmayorga left a comment

Choose a reason for hiding this comment

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

Adding my review to test the release workflow

@gpmayorga gpmayorga mentioned this pull request Dec 20, 2024
Merged
gpmayorga
gpmayorga requested changes Dec 20, 2024
View reviewed changes
Copy link
Collaborator

@gpmayorga gpmayorga left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Sorry for the noise, I needed a PR where I could test the "review approved" trigger and make sure the releas process works. Since this is tagged for release I did my testing here.

Opened a new PR with the CI changes only to be clean and not dependant on when this merges #38

@sophialittlejohn sophialittlejohn enabled auto-merge (squash) January 7, 2025 14:58
@sophialittlejohn sophialittlejohn merged commit 7d2f6c0 into main Jan 8, 2025
6 checks passed
@sophialittlejohn sophialittlejohn deleted the more-reports branch January 8, 2025 15:09
@gpmayorga gpmayorga restored the more-reports branch January 8, 2025 15:13
@gpmayorga gpmayorga deleted the more-reports branch January 8, 2025 15:16
This was referenced Jan 8, 2025
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gpmayorga gpmayorga gpmayorga approved these changes

@onnovisser onnovisser onnovisser approved these changes

Assignees
No one assigned
Labels
alpha Include in alpha release
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@sophialittlejohn @gpmayorga @onnovisser @actions-user