Skip to content

Fix environment variables and script array handling - #10

Open
mavci42 wants to merge 1 commit into
Abstract-Foundation:mainfrom
mavci42:main
Open

Fix environment variables and script array handling#10
mavci42 wants to merge 1 commit into
Abstract-Foundation:mainfrom
mavci42:main

Conversation

@mavci42

@mavci42 mavci42 commented Feb 7, 2025

Copy link
Copy Markdown
  • Environment Variables Handling:

    • Added checks for environment variables to replace undefined values with an empty string.
    • This prevents JSON.stringify(undefined) from returning "undefined".
  • Script Array Filtering:

    • Used filter(Boolean) to remove undefined elements from the script array.
    • This avoids unexpected behavior and ensures a safer configuration.

- Replaced `undefined` environment variables with empty string.
- Filtered `undefined` elements in `script` array using `filter(Boolean)`.
@vercel

vercel Bot commented Feb 7, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
dapp-portal (staging) ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 7, 2025 8:05am

@vercel

vercel Bot commented Feb 7, 2025

Copy link
Copy Markdown

@mavci42 is attempting to deploy a commit to the Abstract Foundation Team on Vercel.

A member of the Team first needs to authorize it.

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.

1 participant