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

feat(wallet): Improve how we're refreshing wallet's ongoing balance #2682

Merged
merged 1 commit into from
Oct 14, 2024

Conversation

rsempe
Copy link
Collaborator

@rsempe rsempe commented Oct 14, 2024

This pull request introduces a new feature to flag wallets that are ready to be refreshed and updates the related logic across the codebase. The most important changes include adding a new ready_to_be_refreshed attribute to the Wallet model, updating the job and service logic to handle this new attribute.

Model Changes:

  • app/models/wallet.rb: Added a new scope ready_to_be_refreshed and a new boolean attribute ready_to_be_refreshed with a default value of false.

Job and Service Logic Updates:

Database Migration:

@rsempe rsempe added the Wallet label Oct 14, 2024
@rsempe rsempe marked this pull request as draft October 14, 2024 10:16
@rsempe rsempe force-pushed the refresh-wallets branch 5 times, most recently from dbad5d8 to e141e57 Compare October 14, 2024 11:44
@rsempe rsempe marked this pull request as ready for review October 14, 2024 11:45
@rsempe rsempe marked this pull request as draft October 14, 2024 12:12
@rsempe rsempe marked this pull request as ready for review October 14, 2024 12:16
@rsempe rsempe force-pushed the refresh-wallets branch 2 times, most recently from 88ca016 to 3620d77 Compare October 14, 2024 12:40
@rsempe rsempe merged commit 46f1b29 into main Oct 14, 2024
6 checks passed
@rsempe rsempe deleted the refresh-wallets branch October 14, 2024 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants