Skip to content

Conversation

@rolodato
Copy link
Contributor

@rolodato rolodato commented Nov 7, 2024

Thanks for submitting a PR! Please check the boxes below:

  • I have added information to docs/ if required so people know about the feature!
  • I have filled in the "Changes" section below?
  • I have filled in the "How did you test this code" section below?
  • I have used a Conventional Commit title for this Pull Request

Changes

Fixes Node.js to use the latest version (v4).

This change depends on Flagsmith/flagsmith-nodejs-client#166 and Flagsmith/flagsmith-nodejs-client#165 to be released.

Committed with --no-verify since I couldn't get prettier to stop breaking my code after reformatting. This breaks this PR and I haven't had time to fix it.

How did you test this code?

Docs built locally, code snippets typechecked with an editor but not actually runtime tested.

"Node.js" instead of "NodeJS"

Node 18 version bump

Don't be cool and use "npm i", --save is also not needed in modern npm

ES6 syntax
@rolodato rolodato requested a review from a team as a code owner November 7, 2024 22:40
@rolodato rolodato requested review from agregoryfs and removed request for a team November 7, 2024 22:40
@vercel
Copy link

vercel bot commented Nov 7, 2024

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

Name Status Preview Comments Updated (UTC)
docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 28, 2024 1:43pm
2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
flagsmith-frontend-preview ⬜️ Ignored (Inspect) Visit Preview Nov 28, 2024 1:43pm
flagsmith-frontend-staging ⬜️ Ignored (Inspect) Visit Preview Nov 28, 2024 1:43pm

@github-actions github-actions bot added the docs Documentation updates label Nov 7, 2024
@rolodato rolodato changed the title Docs/nodejs v4 client docs: Node.js v4 docs Nov 7, 2024
@rolodato rolodato changed the title docs: Node.js v4 docs docs: Node.js v5 docs Nov 28, 2024
Comment on lines +378 to +379
const showButton = flags.isFeatureEnabled('secret_button');
const buttonData = flags.getFeatureValue('secret_button');
Copy link
Contributor

Choose a reason for hiding this comment

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

If we're changing this here, we should also do it here for consistency.

@rolodato rolodato merged commit ada424d into main Nov 28, 2024
10 checks passed
@rolodato rolodato deleted the docs/nodejs-v4-client branch November 28, 2024 18:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants