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

feat(locksmith): API for eventcaster is now async #15039

Merged
merged 13 commits into from
Nov 8, 2024
Merged

Conversation

julien51
Copy link
Member

@julien51 julien51 commented Nov 8, 2024

Description

Issues

Fixes #
Refs #

Checklist:

  • 1 PR, 1 purpose: my Pull Request applies to a single purpose
  • I have commented my code, particularly in hard-to-understand areas
  • I have updated the docs to reflect my changes if applicable
  • I have added tests (and stories for frontend components) that prove my fix is effective or that my feature works
  • I have performed a self-review of my own code
  • If my code involves visual changes, I am adding applicable screenshots to this thread

Release Note Draft Snippet

@cla-bot cla-bot bot added the cla-signed label Nov 8, 2024
@julien51 julien51 changed the title Async eventcaster feat(locksmith): API for eventcaster is now async Nov 8, 2024
@julien51 julien51 requested a review from 0xTxbi November 8, 2024 21:23
@julien51 julien51 marked this pull request as ready for review November 8, 2024 21:23
@julien51 julien51 merged commit 0749772 into master Nov 8, 2024
13 checks passed
@julien51 julien51 deleted the async-eventcaster branch November 8, 2024 21:40
blahkheart pushed a commit to blahkheart/unlock that referenced this pull request Nov 9, 2024
)

* wip

* adding file

* refactor

* more

* setting value from 1password

* wip

* adding async jobs

* cleaned up

* cleanup
clemsos added a commit that referenced this pull request Nov 13, 2024
* - Added proposal to lower change duration
- Modified command in README.md to test proposals locally on fork of base instead of mainnet

* refactor and clean up

* feat(ui): adding another method to advance tabs (#15035)

adding another method to advance

* feature(unlock-app): Migrate Unlock accounts (#15033)

* legacy auth wrapper

* migration page

* migration layout

* update privy config

* legacy hooks and providers

* update dashboard header

* legacy auth context

* add migration compoenents

* update SignInAccount

* update EnterCode component

* update EnterCode component

* update providers

* clean up

* trigger upon successful email verification

* add connectingwaas component

* update SignInAccount

* add legacy hooks

* clean up

* linting

* some cleanup

* more cleanup

* more cleanup

* more work

* better syntax

* update ConnectViaEmail component

* adding another method to advance

* fix accountType

* dedicated page for google sign in

* update password signin component

* update popup component

* update google signin component

* adding migration with code

* update MigrateUserContent component

* removed extra file

* once more

* update google auth flow and usage

* styling

* fixed styling

* update import usage

* updated title

* use privy callback

* add link to direct to dashboard settings

* cleanup

---------

Co-authored-by: Julien Genestoux <[email protected]>

* feat(smart-contracts): add lock upgrade test (#15034)

lock upgrade test

* feat(smart-contracts): emit `PaymentReceipt` event (#14832)

* refactor legacy purcahse function

* basic testing

* update main helper to use logic

* fix lock helper

* add docstrings to interface

* pay referrers

* lint fixes

* fix lint

* fix erc20 keys helper

* emit `PurchaseReceipt` event

* add events

* fire event in extend + add some tests

* test for tip

* rename to `PaymentReceipt`

* fix receipts

* feature(unlock-app): Prompt user to sign out before account migration (#15036)

* update modal component

* add sign out prompt component

* update user migration content

* feature(locksmith): re-enable captcha for staging (#15037)

re-enable captcha

* fix(unlock-app): actually showing the captcha component (#15038)

actually showing the captcha component

* feature(unlock-app): Improve wallet migration with responsiveness, auth, and feedback (#15040)

* responsiveness

* fully authenticate after import & comprehensive user feedback

* fix(unlock-app): using captcha in both places (#15041)

using captcha in both places

* feat(unlock-app): waiting on migration

* feat(unlock-app): creating a waas wallet for debugging purposes

* fix(unlock-app): Fix UI conflict with sign-out prompt during account migration (#15043)

* update migration feedback component

* set and apply migration status

* fix(unlock-app): Fix Google OAuth flow for account migration and add error logging (#15044)

* update waas utility function

* update google sign in component

* update component for google oauth

* fix(unlock-app): Better oauth state handling (#15045)

better popup oauth state handling

* fix(unlock-app): refactor waas util and clean up sessions (#15046)

* refactor keyUtil

* improve google sign in component

* sign out nextauth session upon successful privy login

* cleanup

* feat(locksmith): API for eventcaster is now async (#15039)

* wip

* adding file

* refactor

* more

* setting value from 1password

* wip

* adding async jobs

* cleaned up

* cleanup

---------

Co-authored-by: Julien Genestoux <[email protected]>
Co-authored-by: txbì <[email protected]>
Co-authored-by: Clément Renaud <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant