-
Notifications
You must be signed in to change notification settings - Fork 41
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
Add retry mechanism for partially failed virtual card order #291
Comments
I think reading from local storage is ideal, then from the chain, generally speaking for these sorts of things. |
I think this problem will be solved once we implement reward rollup and permits will be generated on demand. For virtual cards this means when the virtual card is created, a entry in the DB will be created but if there's an error creating a virtual card, you don't write to the DB and everything is as it was. The user can retry the operation until it successfully creates a virtual card. |
What I see is that we are probably going to introduce the virtual card feature before the rollup feature. |
Can someone dm me sandbox API keys? I can't register in Reloadly since I don't have a business email. |
/start |
@zugdev the deadline is at Sat, Oct 19, 4:15 AM UTC |
I think you can use any email provider like proton mail |
It's https://www.reloadly.com/ right? |
Im sure that you can use your own domain's email then. Setting up email forwarding is simple on cloudflare and google domains. |
/start |
! This issue is already assigned. Please choose another unassigned task. |
@gentlementlegen disqualify check |
You can always check the logs: https://github.com/ubiquity-os-marketplace/daemon-disqualifier/actions/runs/11548428245/job/32139825686 |
Can you fix this signature thing across all of our plugins? |
@whilefoo pointed out that it could potentially be because of the use of both the beta and the release that would require different signatures. |
Do you guys have a solution in mind? |
If that is indeed the reason we need to make proper use of environments for the plugins and secrets. I need to check if that's the root cause. |
We could modify the SDK to verify the signature for either prod or dev bot |
/start |
Tip
|
In the virtual card feature, we have two parts in a single transaction to mint a virtual card.
The problem is that the transaction isn't atomic because each step happens in a different system. Step-2 can fail while still leaving the permit amount in the treasury. Why would step-2 fail? There could be one of the many reasons.
Currently, the solution to this problem is manual. It is either a manual refund or manual resending of the card order request to
/post-order
with some parameters (productId, tx hash, country).We can improve this by implementing one of the following solutions.
/post-order
I am inclined towards resending the mint request. One problem is that
/post-order
needs the transaction hash of the permit transfer to the treasury. Sometimes you will have it, and you can store it in the cookies/local storage. But sometimes you will not have it and you will have to read it either from blockchain or ask the user to manually insert the transaction hash to retry minting. I think you can read it from blockchain by looking into permit2 eventsThe goal of this issue is to implement a mechanism to resend the mint request or a better solution if you can think of one.
Since the best-picked virtual card can change in this duration, a better option is to rename the mint button to "Retry Mint" and let the user click first so that user can see they are ordering a different card now.
Please note that the virtual cards feature is present in
beta
branch at the moment. But it can move to development branch anytime. So, keep an eye on that and open a PR accordingly.The text was updated successfully, but these errors were encountered: