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

Make sure LSP url is a valid url when switching #1217

Merged
merged 2 commits into from
Jun 11, 2024
Merged

Conversation

benthecarman
Copy link
Collaborator

No description provided.

@TonyGiorgio
Copy link
Contributor

still doesn't work

Screenshot 2024-06-10 at 8 00 35 PM

@benthecarman benthecarman force-pushed the some-lsp-fixes branch 2 times, most recently from fb9221f to f6a8bf7 Compare June 11, 2024 02:21
@benthecarman
Copy link
Collaborator Author

Main purpose of this wasn't to be able to get people out of this state but to prevent it from happening. Added some trim()s everywhere that should help get your wallet unstuck

@TonyGiorgio
Copy link
Contributor

I used a new wallet and still got into this state.

@benthecarman
Copy link
Collaborator Author

Think this should prevent everywhere, otherwise plz lmk how you reproduce

@TonyGiorgio TonyGiorgio merged commit f00bc5e into master Jun 11, 2024
9 checks passed
@TonyGiorgio TonyGiorgio deleted the some-lsp-fixes branch June 11, 2024 16:38
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.

2 participants