Skip to content

Conversation

@floganz
Copy link
Contributor

@floganz floganz commented Dec 4, 2025

Context

We’re introducing the priority field to wallets to support custom execution or display ordering. This PR builds on the base work from #4391, which added the field to the Wallet model and exposed it via the API.

The goal of this PR is to implement the logic that uses the priority value when handling wallets — for example, when determining which wallets to apply or process first.

Description

This PR includes:
• Logic to order wallets by priority (and created_at as a secondary key).
• Ensures active_wallets_in_application_order returns wallets in the correct priority sequence.
• Updates any internal wallet processing to respect the defined order.

This builds directly on the foundation laid in PR #4391 and assumes the priority field and its validations are already in place.

@floganz floganz self-assigned this Dec 4, 2025
@floganz floganz force-pushed the feat-multiple-wallets branch from 3d22587 to ae0327e Compare December 4, 2025 15:47
@floganz floganz marked this pull request as ready for review December 4, 2025 15:59
Copy link
Contributor

@annvelents annvelents left a comment

Choose a reason for hiding this comment

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

left couple of small comments

Copy link
Collaborator

@vincent-pochet vincent-pochet left a comment

Choose a reason for hiding this comment

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

Looks good 🚀

@floganz floganz force-pushed the feat-multiple-wallets branch from 806db97 to 4d6ee86 Compare December 8, 2025 14:23
@floganz floganz merged commit 05587e4 into main Dec 9, 2025
10 checks passed
@floganz floganz deleted the feat-multiple-wallets branch December 9, 2025 08:07
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.

5 participants