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 node.js only package in client issue #130

Merged
merged 1 commit into from
Sep 19, 2024
Merged

fix node.js only package in client issue #130

merged 1 commit into from
Sep 19, 2024

Conversation

joewagner
Copy link
Contributor

There seems to have been a regression in the table nft animation_url app where the dotenv packages was being used on the client. This caused the app to fail to load correctly.
dotenv had been added to facilitate loading the Wallet Connect (now called Reown) project id.

This PR removes the use of dotenv and uses a react custom env var. If that env is not available during build, a default Reown project id has been added as a backup.

@joewagner joewagner self-assigned this Sep 19, 2024
@joewagner joewagner marked this pull request as ready for review September 19, 2024 02:54
@joewagner joewagner merged commit 2b0f246 into main Sep 19, 2024
5 checks passed
@joewagner joewagner deleted the joe/hotfix-1 branch September 19, 2024 03:13
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