Skip to content

fix: adds Svelte and Next.js #389

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

HashMapsData2Value
Copy link
Contributor

@HashMapsData2Value HashMapsData2Value commented Jul 16, 2025

Upgrades liquid auth use-wallet-client version to make Svelte work.

ETA: after trying with Next.js and discovering that it also works, I'm re-adding it.

@HashMapsData2Value HashMapsData2Value marked this pull request as draft July 16, 2025 09:11
@HashMapsData2Value
Copy link
Contributor Author

HashMapsData2Value commented Jul 16, 2025

Just need to address a CORS issue in the Liquid Auth backend, which you experience when you try to press "Disconnect" in one of the example apps running on localhost. Anyone hosting their own Liquid Auth backend with use-wallet on a page, all on their own domain, should not be experiencing that problem. Furthermore, it is possible to disconnect by refreshing the page.

@HashMapsData2Value HashMapsData2Value changed the title fix: adds svelte fix: adds Svelte and Next.js Aug 6, 2025
@HashMapsData2Value HashMapsData2Value marked this pull request as ready for review August 6, 2025 12:03
@drichar
Copy link
Collaborator

drichar commented Aug 16, 2025

@HashMapsData2Value as we discussed, Liquid Auth as been removed temporarily until the peer dependency issues are sorted out. Since they affect production dApp builds even if the Liquid Auth provider is not configured, this is a necessary step until the issue(s) are resolved.

I'll keep this PR open so it can be merged after the Liquid Auth provider has been restored.

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