Skip to content

Universal checkout #251

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.

Sign up for GitHub

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

Jump to bottom
Open
dcposch wants to merge 225 commits into
base: master
Choose a base branch
from
Open

Universal checkout #251

dcposch wants to merge 225 commits into from

Conversation

@dcposch
Copy link

@dcposch dcposch commented Feb 10, 2025

Summary

See DIP-64.

  • Better checkout, using Daimo Pay.
  • Automatic refunds, using Peanut.

@mikulas-mrva
Adding web3modal as a build package for now, changing multiple receiv…
9236481 
…er mode to single receiver, deleting check for available receiving addresses
@mikulas-mrva
removing wallet upload models and views, adding wallet connection js,…
5d62b24 
… transaction details view in progress
@mikulas-mrva
Added SignedMessage model, view and serializer to send payment detail…
01a04f1 
…s to the FE
@mikulas-mrva
Finished the FE code outline, added Permissions to allow POST and GET…
b106f91 
… requests for anonymous users
@mikulas-mrva
Switching to the correect way of signing a message with Metamask
dcdb640
@mikulas-mrva
Changing data signing to typed data
7951816
@mikulas-mrva
Fixing data signing
40f9028
@mikulas-mrva
Confirming payments, deleting unused permissions module
f11ad3b
@mikulas-mrva
Improving user flow of the payment process, displaying a message when…
f6d1579 
… the user should sign a message and confirm a transaction
@mikulas-mrva
Missing migration
6bf8d96
@mikulas-mrva
Changing raw_message to text field to circumvent field length limitat…
cba6705 
…ions
@mikulas-mrva
Moving retry triggers from inline to js, adding transaction confirmat…
c0bd7e0 
…ion text for the user.
@mikulas-mrva
Handling user dialogue cancellations gracefully.
eefabbb
@mikulas-mrva
Do not hide success message if the transaction was submitted successf…
c958df3 
…ully
@mikulas-mrva
chain_id should be larger than uint8 to accomodate all chains
0f531d0
@mikulas-mrva
Fixing a typo, using signTypedData_v4 for signing
1a6a8df
@mikulas-mrva
Network name should not be uppercase in _RPC_URL
3760617
@mikulas-mrva
Extending chain id field
24ad2d7
@mikulas-mrva
Bug fixes
7fc5597
@mikulas-mrva
Don't let the user pay for multiple order payments wwithin one order
5b125b6
@mikulas-mrva
Disabling refunds
964eacc
@mikulas-mrva
fixes:
1e71c56 
* Adding ERC20 ABI view for ERC20 payments
* making sure only one order_payment can be paid for
* no longer disabling the "Connect Wallet and pay" button
@mikulas-mrva
Confirming ERC20 payments
e47ced1
@mikulas-mrva
Don't confirm payments based on blocks that are too young
12cb449
@mikulas-mrva
Display transaction hash on refresh, display transaction details if t…
ead478f 
…here's no hash yet.
@mikulas-mrva
Add ERC20 balance check to prevent most failing transactions to be su…
ba270eb 
…bmitted
@mikulas-mrva
Automatically confirming payments, ignoring payments that didn't go t…
39f197f 
…hrough
@mikulas-mrva
Preventing a payment to go through in case of insufficient balance er…
a4329ec 
…ror, reloading the page on account change instead of triggering the logic immediately
@mikulas-mrva
Only display "please sign a message dialogue" in case it's about to p…
3746b18 
…op up in the browser.
@mikulas-mrva
Only display "please confirm a transaction" right before a transactio…
a6cfe42 
…n dialog is about to pop up
@dcposch dcposch force-pushed the master branch 2 times, most recently from 632a61d to fb7c8fb Compare April 4, 2025 07:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@dcposch @mikulas-mrva @kvbik @lassejaco @andrewliu08