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

Inlining checkout & Single receiver mode #215

Open
wants to merge 84 commits into
base: master
Choose a base branch
from

Conversation

mikulas-mrva
Copy link
Collaborator

@mikulas-mrva mikulas-mrva commented May 22, 2022

Fixes multiple open issues

  • Implementing single receiver mode
  • Inlining Web3checkout
  • CI timeout in pip install !207
  • Adding Goerli testnet

@mikulas-mrva mikulas-mrva force-pushed the inlining-checkout branch 5 times, most recently from 6989098 to d38bc4c Compare July 14, 2022 14:16
@mikulas-mrva mikulas-mrva force-pushed the inlining-checkout branch 3 times, most recently from 94cae60 to bef66bb Compare July 16, 2022 17:53
pretix_eth/exporter.py Outdated Show resolved Hide resolved
pretix_eth/management/commands/confirm_payments.py Outdated Show resolved Hide resolved
# native asset
if token.IS_NATIVE_ASSET:
try:
transaction_details = w3.eth.getTransaction(signed_message.transaction_hash)
Copy link
Contributor

Choose a reason for hiding this comment

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

why is this not getTransactionReceipt?

pretix_eth/management/commands/confirm_payments.py Outdated Show resolved Hide resolved
pretix_eth/models.py Outdated Show resolved Hide resolved
…er mode to single receiver, deleting check for available receiving addresses
… the user should sign a message and confirm a transaction
@mikulas-mrva mikulas-mrva force-pushed the inlining-checkout branch 2 times, most recently from 58f394a to 7b87e35 Compare September 30, 2022 16:14
@mikulas-mrva mikulas-mrva changed the title WIP: Inlining checkout Inlining checkout & Single receiver mode Dec 5, 2022
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.

3 participants