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

DIP-32: Email Welcome Packs with Testnet Tokens for Workshops at DEVCon(nect) #123

Closed
wants to merge 2 commits into from

Conversation

0xkkonrad
Copy link

Description

Related DIP issue

Checklist

  • [x ] Correct markdown (e.g. headers with ##, header surrounded by ---)
  • [ x] syntax check
  • [x ] filename is DIP-X.md (with X being the correct DIP number)
  • [ x] header contains (in this order)
    • [x ] DIP
    • [x ] Title
    • [x ] Status
    • [x ] Themes
    • [x ] Tags
    • [x ] Authors
    • [x ] Resources Required (optional)
    • [x ] Discussion (with link to devcon forum)
    • [x ] Created (in the format year-month-day e.g 2020-4-20)
  • [ x] contains these sections:
    • [x ] Abstract
    • [ x] Rationale
    • [ x] Implementation
    • [ x] Operational Requirements & Ownership
  • [ x] PR title prefixed with DIP-X

Copy link
Member

@ligi ligi left a comment

Choose a reason for hiding this comment

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

Thanks for the proposal - just some small nits - formally only the Rationale is missing
We will discuss it later this week (Thursday)

DIPs/DIP-32.md Outdated Show resolved Hide resolved
DIPs/DIP-32.md Outdated Show resolved Hide resolved
DIPs/DIP-32.md Outdated Show resolved Hide resolved
DIPs/DIP-32.md Outdated Show resolved Hide resolved
DIPs/DIP-32.md Show resolved Hide resolved
DIPs/DIP-32.md Outdated Show resolved Hide resolved
small fixes reviewed

Co-authored-by: ligi <ligi@ligi.de>
@RiverRuby
Copy link
Contributor

It may also be worth a collaboration with emailwallet.org team, a group funded by EF's PSE! This way the ticket email can directly be the receiver of the test token.

@ligi
Copy link
Member

ligi commented Oct 12, 2023

@vb7401 can you elaborate how the collaboration would work on a technical level?

@RiverRuby
Copy link
Contributor

RiverRuby commented Oct 12, 2023

I'm not a full expert, just a friend/fan of that team haha. From my understanding, they are building a smart contract wallet for arbitrary email addresses email@domain.com, where transferring tokens is gated by sending an email from email@domain.com with the specified action. This works because each email comes with a DKIM signature from its domain.com mailserver, which can then be verified on-chain cheaply using a ZKP, from which the action specified (sending money, sending an NFT) can be authorized.

It may be too early stage for them to support the specific types of test tokens this proposal needs, but roughly I am imagining that these tokens can directly be given to the smart contract wallets for each of the pretix emails instead of doing a flow with a claim link + sending them to another wallet. From my understanding of the protocol, this can be done even if the email hasn't set up a wallet yet! The best person to contact is https://twitter.com/yush_g, one of the leads on the project.

In general, I am very bullish on emailwallet.org long-term being the right medium for "token airdrop to emails that have a ticket" solutions, especially in communities that are less web3 focused/pilled!

@ligi
Copy link
Member

ligi commented Oct 12, 2023

I would need to dig into email-wallet from the surface I have concerns wrt centralisation and readyness - just wanted to try it out and got this:
image

So I think for DEVConnect we should go with peanut

@RiverRuby
Copy link
Contributor

Yep, they're working on shipping a lot of the functionality for Devconnect, so it might be too difficult to make everything work out. Perhaps for a future Devcon 😃🤞

@ligi
Copy link
Member

ligi commented Nov 11, 2023

Sorry but we went with the PCDPass faucet solution in the end: https://twitter.com/_pk910_/status/1721472568765345955

Your solution is great - but we want to encourage users to use PCDPass

@ligi ligi closed this Nov 11, 2023
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