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

fix: login info lost for mining #1426

Merged
merged 22 commits into from
Jan 24, 2025
Merged

fix: login info lost for mining #1426

merged 22 commits into from
Jan 24, 2025

Conversation

PanchoBubble
Copy link
Collaborator

@PanchoBubble PanchoBubble commented Jan 22, 2025

  • Moves start/stop mining to its own file
  • Removes client side persist of airdrop user token and saves it on the be side
  • Adds restart mm proxy logic on the be side when user tokens are updated

@brianp
Copy link
Collaborator

brianp commented Jan 23, 2025

During testing its noted my airdrop was logged out again when running this branch. Is there an expectation of persisting existing logins or should it have been invalidated?

Logins between restarts on this branch seem fine. Logins from any other branches are fine on those branches.
Logins from other branches aren't carried over to this branch.

@brianp brianp merged commit dc8b962 into main Jan 24, 2025
11 checks passed
@brianp brianp deleted the fix-login-info-lost-for-mining branch January 24, 2025 13:39
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.

4 participants