Skip to content

doc(technologies.server.java): use proper variable#582

Closed
dancer1325 wants to merge 1 commit into
open-feature:mainfrom
dancer1325:fix/use_right_variable
Closed

doc(technologies.server.java): use proper variable#582
dancer1325 wants to merge 1 commit into
open-feature:mainfrom
dancer1325:fix/use_right_variable

Conversation

@dancer1325

Copy link
Copy Markdown

This PR

  • Fix to use the proper variable

Related Issues

  • NO issue ticket. Directly created the PR, for this super minor fix.

Notes

How to test

  • Check the fragment code defined to set transactionContext -- Link --
// adding userId to transaction context
OpenFeatureAPI api = OpenFeatureAPI.getInstance();
Map<String, Value> transactionAttrs = new HashMap<>();
transactionAttrs.put("userId", new Value("userId"));
EvaluationContext transactionCtx = new ImmutableContext(transactionAttrs);
api.setTransactionContext(apiCtx);

@netlify

netlify Bot commented Jun 1, 2024

Copy link
Copy Markdown

Deploy Preview for openfeature ready!

Name Link
🔨 Latest commit 81cf41b
🔍 Latest deploy log https://app.netlify.com/sites/openfeature/deploys/665b1accdd249c00085d9326
😎 Deploy Preview https://deploy-preview-582--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.

Signed-off-by: alfredo-toledano <alfredo.toledano@allianzdirect.es>
@beeme1mr

beeme1mr commented Jun 4, 2024

Copy link
Copy Markdown
Member

Hey @dancer1325, good catch! These docs are automatically generated from the Java SDK readme. I've ported this change there and reran the generator. Thanks for letting us know about this issue.

#580

@beeme1mr beeme1mr closed this Jun 4, 2024
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