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

Fix variable name in a code snippet #573

Closed
wants to merge 1 commit into from

Conversation

gavvvr
Copy link

@gavvvr gavvvr commented May 25, 2024

This PR

  • Fixes variable name in Transaction Context Java code snippet

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

netlify bot commented May 25, 2024

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
Copy link
Member

beeme1mr commented Jun 4, 2024

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
Copy link
Member

beeme1mr commented Jun 4, 2024

#580

@beeme1mr beeme1mr closed this Jun 4, 2024
@beeme1mr
Copy link
Member

beeme1mr commented Jun 4, 2024

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