-
Notifications
You must be signed in to change notification settings - Fork 15
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
Update data apps default secrets #689
Conversation
@pepamartinec can you please review my proposed changes? Did I describe it correctly, please? Thank you. |
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.
Z pohledu usera to asi bude v pohode.
Kdybych chtel byt pintlich, tak pokud by se ta sekce mela jmenovat Default ENV vars
, mela by toho obsahovat vic (tech envu je vic, museli bysme si je asi projit, co muze byt zajimavy pro usera). A nebo bych to cely prejmenoval na Access Storage from Data App
a trochu podle toho upravil textaci, at je to proste navedeni jednoduse, jak se napojit na Storage (to je konec koncu asi to, co bych jako user v tom navodu hledal)
@pepamartinec změnila jsem úvod na toto: Access Storage from Data AppBy default, the Keboola Streamlit Docker image includes two environment variables, which can be easily used to access keboola storage from your application: Může to tak být? |
@hhanova can you please check it as well? Thank you. |
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.
Mozna jeste takovy detail
Jira issue(s): PROOF-XXX
Changes: