-
Notifications
You must be signed in to change notification settings - Fork 729
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
[SDKs] DAPR_API_TOKEN env variable #3921
Comments
@msfussell ok to close this? or did we need this in other SDKs? |
I just created an issue for it in python-sdk dapr/python-sdk#653. |
Here's the PR: dapr/python-sdk#654. It's a very small update. |
@elena-kolevska - Does this need to be done with other SDKs? |
The only code change we did was in the JS sdk, all the other ones already had this feature, but not all of them had the docs properly updated. Here's a list:
@cicoyle would you be able to help with the java-sdk docs for this? (I've never worked with this sdk and don't have the environment set up to test). I would suggest copying the Initialising the client section from the python-sdk docs and adapting the snippets to Java. |
I added this PR for the Java sdk, but since I'm not very familiar with the code I'm gonna rely on Cassie or someone else to confirm it's correct. |
oops, I missed your ping on here. thanks @elena-kolevska |
I think we can now consider this done. |
@elena-kolevska sounds good - closing as resolved! |
What content needs to be created or modified?
Check and make sure this is documented across all the SDKs (open PRs in sdk repos)
Where should the new material be placed?
SDK docs
The associated pull request from dapr/dapr, dapr/components-contrib, or other Dapr code repos
dapr/js-sdk#547
The text was updated successfully, but these errors were encountered: