volto icon indicating copy to clipboard operation
volto copied to clipboard

Document environment variables

Open Manas-Kenge opened this issue 1 year ago • 7 comments

Fixes #4581 #5319

@tiberiuichim I updated the docs as per my understanding, was I supposed to create a separate section for it? I looked at the Razzle docs for reference.

Manas-Kenge avatar Feb 05 '24 18:02 Manas-Kenge

Deploy Preview for volto ready!

Name Link
Latest commit 7bd7ce80dc417838acfa74bc21bd58d69df357c7
Latest deploy log https://app.netlify.com/sites/volto/deploys/65ecbb2eeb260c0008a9a67e
Deploy Preview https://deploy-preview-5735--volto.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.

netlify[bot] avatar Feb 05 '24 18:02 netlify[bot]

Deploy Preview for plone-components canceled.

Name Link
Latest commit 7bd7ce80dc417838acfa74bc21bd58d69df357c7
Latest deploy log https://app.netlify.com/sites/plone-components/deploys/65ecbb2e1eb9c900075d0dcb

netlify[bot] avatar Feb 05 '24 18:02 netlify[bot]

@Manas-Kenge please hold on this pull request, per https://github.com/plone/volto/issues/4581#issuecomment-1928577566.

stevepiercy avatar Feb 06 '24 00:02 stevepiercy

Ok, no problem.

Manas-Kenge avatar Feb 06 '24 09:02 Manas-Kenge

@Manas-Kenge I overhauled the file in https://github.com/plone/volto/pull/5736. You may rebase your work on main and resume work on this issue.

stevepiercy avatar Mar 07 '24 09:03 stevepiercy

I referred the Razzle documentation https://razzlejs.org/docs/environment-variables for this. Should I just remove process.env then?

Manas-Kenge avatar Mar 09 '24 18:03 Manas-Kenge

@Manas-Kenge The Razzle documentation is describing environment variables that are set automatically by Razzle and can be accessed by application code.

Our docs are trying to describe environment variables that can be set manually from outside the application to influence how it runs.

davisagli avatar Mar 09 '24 18:03 davisagli