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

186-Store-PoolSnapshot-data-on-epoch-close-linked-to-Epoch #223

Merged

Conversation

filo87
Copy link
Collaborator

@filo87 filo87 commented Jul 8, 2024

closes #186
closes #221

BREAKING CHANGES:
Aggregated NAV queries in UI should now be constructed like this:

{
  snapshotPeriods(filter: { weekDay: { equalTo: 1 } }, orderBy: START_DESC, first: 1000) {
    nodes {
      id
      poolSnapshots {
        aggregates {
          sum {
            normalizedNAV
          }
        }
      }
    }
  }
}

Filed periodStart is DEPRECATED on all snapshots. Substituted with a foreigh key periodId to a SnapshotPeriod entity.

filo87 added 2 commits July 8, 2024 18:26
BREAKING CHANGE: `periodStart` deprecated on Snapshots. Substituted by `period` foreign key
@filo87 filo87 self-assigned this Jul 8, 2024
@filo87 filo87 requested a review from hieronx July 9, 2024 08:48
@filo87 filo87 enabled auto-merge (squash) July 10, 2024 08:58
@filo87 filo87 merged commit fb7c486 into main Jul 10, 2024
3 checks passed
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.

Track periods with dedicated entity Store PoolSnapshot data on epoch close, linked to Epoch
2 participants