Skip to content
This repository was archived by the owner on Feb 3, 2025. It is now read-only.

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

Merged
AnthonyRonning merged 2 commits into
masterfrom
some-lsp-fixes
Jun 11, 2024
Merged

Make sure LSP url is a valid url when switching#1217
AnthonyRonning merged 2 commits into
masterfrom
some-lsp-fixes

Conversation

@benthecarman

Copy link
Copy Markdown
Collaborator

No description provided.

@AnthonyRonning

Copy link
Copy Markdown
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
Copy Markdown
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

@AnthonyRonning

Copy link
Copy Markdown
Contributor

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

@benthecarman

Copy link
Copy Markdown
Collaborator Author

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

@AnthonyRonning AnthonyRonning merged commit f00bc5e into master Jun 11, 2024
@AnthonyRonning AnthonyRonning deleted the some-lsp-fixes branch June 11, 2024 16:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants