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.

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

feat(user): transfer a wallet from anonymous user to a logged in one #318

Merged
merged 1 commit into from
Aug 20, 2024
Merged

feat(user): transfer a wallet from anonymous user to a logged in one #318

merged 1 commit into from
Aug 20, 2024

Conversation

ygrishajev
Copy link
Contributor

The scenario this PR is addressing:

  1. there is a registered user that has no managed wallet
  2. the one goes to the console without being logged it
  3. starts trial as an anonymous users (and say, creates a deployment)
  4. logs in

In this case the anonymous user managed wallet (and the deployment along with it) is transferred to the given already registered user.

Also this PR adds timestamps to user and it's wallet which are really useful when debugging.

@ygrishajev ygrishajev requested review from Redm4x and baktun14 and removed request for Redm4x August 19, 2024 14:35
@ygrishajev ygrishajev merged commit edf3d3a into main Aug 20, 2024
5 checks passed
@ygrishajev ygrishajev deleted the feature/user branch August 20, 2024 10:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@baktun14 baktun14 baktun14 approved these changes

@Redm4x Redm4x Awaiting requested review from Redm4x

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ygrishajev @baktun14