Skip to content

Fix variable name in a code snippet#573

Closed
gavvvr wants to merge 1 commit into
open-feature:mainfrom
gavvvr:patch-1
Closed

Fix variable name in a code snippet#573
gavvvr wants to merge 1 commit into
open-feature:mainfrom
gavvvr:patch-1

Conversation

@gavvvr

@gavvvr gavvvr commented May 25, 2024

Copy link
Copy Markdown

This PR

  • Fixes variable name in Transaction Context Java code snippet

Signed-off-by: Kirill Gavrilov <gavvvr@users.noreply.github.com>
@netlify

netlify Bot commented May 25, 2024

Copy link
Copy Markdown

Deploy Preview for openfeature ready!

Name Link
🔨 Latest commit 8d4cb4b
🔍 Latest deploy log https://app.netlify.com/sites/openfeature/deploys/6651dd9417c58a0008d0480b
😎 Deploy Preview https://deploy-preview-573--openfeature.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@beeme1mr

beeme1mr commented Jun 4, 2024

Copy link
Copy Markdown
Member

Hey @gavvvr, sorry for the delay! Good catch and thanks for opening a PR. These files are automatically generated from the Java SDK readme. Could you please make the change there? Thanks!

@beeme1mr

beeme1mr commented Jun 4, 2024

Copy link
Copy Markdown
Member

#580

@beeme1mr beeme1mr closed this Jun 4, 2024
@beeme1mr

beeme1mr commented Jun 4, 2024

Copy link
Copy Markdown
Member

hey @gavvvr, I saw that there were a few other issues in the Java readme so I went ahead and addressed them all in one PR. Thanks for reporting the issue and sorry again for the delayed review 🙇

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