Skip to content

feat: add support for Next env vars #842

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

Merged
merged 3 commits into from
Nov 29, 2021
Merged

feat: add support for Next env vars #842

merged 3 commits into from
Nov 29, 2021

Conversation

ascorbic
Copy link
Contributor

@ascorbic ascorbic commented Nov 25, 2021

Summary

Adds support for Next env var files

Test plan

Check the following pages for the string "Hello Production"

  1. https://deploy-preview-842--netlify-plugin-nextjs-demo.netlify.app/getStaticProps/env
  2. https://deploy-preview-842--netlify-plugin-nextjs-demo.netlify.app/api/hello
  3. https://deploy-preview-842--netlify-plugin-nextjs-demo.netlify.app/getServerSideProps/1

Relevant links (GitHub issues, Notion docs, etc.) or a picture of cute animal

image

Fixes #823

@github-actions github-actions bot added the type: feature code contributing to the implementation of a feature and/or user facing functionality label Nov 25, 2021
Release-As: 4.0.0-beta.12
@ascorbic ascorbic requested a review from tiffafoo November 25, 2021 16:10
@ascorbic ascorbic self-assigned this Nov 25, 2021
@@ -26,6 +26,8 @@
"publish:test": "npm test",
"test": "run-s build build:demo test:jest",
"test:jest": "jest",
"test:jest:update": "jest --updateSnapshot",
"test:update": "run-s build build:demo test:jest:update",

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[sand] we might want to use actual objects, this makes it too easy to run this and ignore snapshots and captures implementation details. It might be a huge object but at least the update will be more deliberate

Copy link

@tiffafoo tiffafoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

neato bonito

@kodiakhq kodiakhq bot merged commit 24fd88a into main Nov 29, 2021
@kodiakhq kodiakhq bot deleted the mk/env-vars branch November 29, 2021 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge type: feature code contributing to the implementation of a feature and/or user facing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support Next.js Environment Variables
2 participants