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

Ch4: Deployment - Updating Environment variable section #6970

Merged
merged 2 commits into from
Dec 2, 2022

Conversation

MrOnosa
Copy link
Contributor

@MrOnosa MrOnosa commented Nov 27, 2022

Netlify has changed how Environment Variables are set. They are no longer nested inside Build & Deploy.

Netlify has changed how Environment Variables are set. They are no longer nested inside Build & Deploy.
Copy link
Collaborator

@pantheredeye pantheredeye left a comment

Choose a reason for hiding this comment

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

Thanks for the update, @MrOnosa! I have reviewed the Env vars section, and all seems well there with Netlify's new Env. Var section. I have not confirmed the change with "Site is live" to "Deployment failed". If noone gets to it prior, I will look to get that confirmed this week. Thanks again!

@MrOnosa
Copy link
Contributor Author

MrOnosa commented Nov 27, 2022

Thank you for double-checking everything. Always appreciated. For context, this is what it shows for me.
image

@cannikin
Copy link
Member

Thanks so much for this PR!

@cannikin cannikin added the release:docs This PR only updates docs label Dec 2, 2022
@cannikin
Copy link
Member

cannikin commented Dec 2, 2022

@cannikin
Copy link
Member

cannikin commented Dec 2, 2022

Looks great, thanks for the fix!

@cannikin cannikin merged commit 0dff471 into redwoodjs:main Dec 2, 2022
@redwoodjs-bot redwoodjs-bot bot added this to the next-release milestone Dec 2, 2022
@redwoodjs-bot
Copy link

redwoodjs-bot bot commented Dec 2, 2022

🔔 @jtoar, @Tobbe—I couldn't cherry pick this one. If you want it in the next release, you'll have to cherry pick it manually.

@MrOnosa MrOnosa deleted the ch4-deployment-update branch December 2, 2022 17:08
dac09 added a commit that referenced this pull request Dec 6, 2022
…xperimental-vite-optin

* 'main' of github.com:redwoodjs/redwood: (26 commits)
  Compress v1 (1.0-1.5) docs to v1.x (#7025)
  Ch4: Deployment - Updating Environment variable section (#6970)
  fix(deps): update dependency systeminformation to v5.16.0 (#7011)
  fix(deps): update graphqlcodegenerator monorepo (#7005)
  fix(deps): update dependency concurrently to v7.6.0 (#7006)
  fix(deps): update dependency mini-css-extract-plugin to v2.7.1 (#7007)
  fix(deps): update dependency eslint to v8.28.0 (#7008)
  fix(deps): update dependency prettier to v2.8.0 (#7009)
  fix(deps): update dependency react-hook-form to v7.40.0 (#7010)
  chore(deps): update dependency lerna to v6.1.0 (#6993)
  fix(deps): update dependency @fastify/http-proxy to v8.4.0 (#6995)
  fix(deps): update prisma monorepo to v4.7.0 (#6999)
  fix(deps): update dependency jest-watch-typeahead to v2.2.1 (#6991)
  fix(deps): update dependency listr2 to v5.0.6 (#6989)
  chore(deps): update dependency cypress to v11.2.0 (#6992)
  chore(deps): update dependency nx to v15.2.4 (#6994)
  fix(deps): update dependency msw to v0.49.1 (#6996)
  auth-firebase-* (#6987)
  fix(cli): Handle importing optional setup auth commands (#6997)
  fix(deps): update dependency eslint-plugin-react to v7.31.11 (#6990)
  ...
@jtoar jtoar modified the milestones: next-release, v4.0.0 Dec 8, 2022
jtoar pushed a commit that referenced this pull request Dec 8, 2022
* Ch4 - Updating Environment variable section

Netlify has changed how Environment Variables are set. They are no longer nested inside Build & Deploy.

* Ch4- Deployment Updating image to show new netlify UI
@jtoar jtoar modified the milestones: v4.0.0, next-release, v3.7.0 Dec 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release:docs This PR only updates docs
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants