-
Notifications
You must be signed in to change notification settings - Fork 4
feat(user-token): Add Web UI token to environment variables #237
Conversation
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://black-sand-0b0e2a903-237.westeurope.3.azurestaticapps.net |
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://black-sand-0b0e2a903-237.westeurope.3.azurestaticapps.net |
str-anger
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As mentioned in the other PR comment, this may probably be a neater change.
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://black-sand-0b0e2a903-237.westeurope.3.azurestaticapps.net |
| my_env["api_token"] = user_info.token | ||
| my_env["aqueduct_url"] = extension.aqueduct_url | ||
| if extension.aqueduct_api_token is not None: | ||
| my_env["API_TOKEN"] = extension.aqueduct_api_token |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I hope these 2 different names (lower and upper case) don't create any problem. If you can make them consistent, this may look neater.
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://black-sand-0b0e2a903-237.westeurope.3.azurestaticapps.net |
Changes