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

Move entity storage to the Orchestration #110

Merged
merged 34 commits into from
Feb 15, 2024

Conversation

wcalderipe
Copy link
Collaborator

@wcalderipe wcalderipe commented Feb 14, 2024

This PR introduces several changes to move the entity storage from the authz to the orchestration application.

Disclaimer

  • The massive PR felt right to delivery an end-to-end solution; nonetheless, my bad.
  • I have aggressively moved things from one place to another without thinking too much about how they're designed.

Changelog (at least what matters)

  • Moved entity management from the authz to the orchestration application.
  • Changed many DTOs to improve Swagger documentation.
  • Replaced mock data by known personas and prefixed addresses.
  • Configured @nestjs/swagger/plugin to automatically annotate DTOs.

Next

@wcalderipe wcalderipe self-assigned this Feb 14, 2024
@wcalderipe wcalderipe marked this pull request as ready for review February 15, 2024 15:11
@wcalderipe wcalderipe merged commit 26d4717 into main Feb 15, 2024
4 checks passed
@wcalderipe wcalderipe changed the title Entity storage Move entity storage to the Orchestration Feb 15, 2024
@wcalderipe wcalderipe deleted the feature/forward-create-user-request branch March 5, 2024 14:17
mattschoch pushed a commit that referenced this pull request Jun 19, 2024
wcalderipe pushed a commit that referenced this pull request Feb 5, 2025
* removed operations, scope sync can partially succeed

* refactor bitgo and fireblocks

* tests fixed for new interface

* removed unused services imports

* adding logs to the wallet bulkUpsert method

* networkMap is a map, addressed namings, used constants instead of numbers

* namings, complete function handles partial success for scoped sync service

* lock sync when there's already an in progresss sync for this connection

* Trying larger runner

* fixed the network caching, and the lock on scope sync

* Revert "Trying larger runner"

This reverts commit bda1bb6b3154ae3f71740bbbcf7d0aca263649dc.

* raw account errors are more precise

* fireblocks raw accounts building use external network id

* fixed reads for upserts outside of tx scope

* exists take the status as a parameter

* removed unused function

---------

Co-authored-by: Matt Schoch <[email protected]>
wcalderipe pushed a commit that referenced this pull request Feb 5, 2025
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.

1 participant