-
-
Notifications
You must be signed in to change notification settings - Fork 856
Docs: Add deprecation warning for src/config.js (Volto 18+) #7756
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
base: main
Are you sure you want to change the base?
Docs: Add deprecation warning for src/config.js (Volto 18+) #7756
Conversation
|
Caution The Volto Team has suspended its review of new pull requests from first-time contributors until the release of Plone 7, which is preliminarily scheduled for the second quarter of 2026. Thanks for submitting your first pull request! You are awesome! 🤗 If you haven't done so already, read Welcome to the Plone community! 🎉 |
|
@gummybearansh you need to sign the Plone Contributor Agreement to merge this pull request. Learn about the Plone Contributor Agreement: https://plone.org/foundation/contributors-agreement If you have already signed the agreement, please allow a week for your agreement to be processed. If after a week you have not received an invitation, then please contact agreements@plone.org. |
stevepiercy
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.
Thanks for your contribution! I've made some suggestions.
|
@gummybearansh you need to sign the Plone Contributor Agreement to merge this pull request. Learn about the Plone Contributor Agreement: https://plone.org/foundation/contributors-agreement If you have already signed the agreement, please allow a week for your agreement to be processed. If after a week you have not received an invitation, then please contact agreements@plone.org. |
Thanks for the detailed review and the guidance on MyST directives! I have updated the file to:
I will open a separate PR shortly to add the .vscode/settings.json file as you suggested. Ready for another look when you have a moment. |
stevepiercy
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.
LGTM, but I want a technical review from the Volto Team before it's merged.
Also before merging, we need your permission to publish your work. See https://6.docs.plone.org/contributing/#sign-and-return-the-plone-contributor-agreement.
|
Also it needs a change log entry per https://6.docs.plone.org/contributing/#change-log-entry |
|
@gummybearansh you need to sign the Plone Contributor Agreement to merge this pull request. Learn about the Plone Contributor Agreement: https://plone.org/foundation/contributors-agreement If you have already signed the agreement, please allow a week for your agreement to be processed. If after a week you have not received an invitation, then please contact agreements@plone.org. |
|
I have already signed and submitted the form. I am currently waiting for the acceptance confirmation I have added the changelog entry. Standing by for Volto team technical reivew. |
Description
Updates the configuration registry documentation to reflect that the project configuration approach using
src/config.jsis deprecated in Volto 18+ in favor of theconfig/folder structure.Related Issue
Closes #7601
Changes
{warning}block todocs/source/configuration/how-to.mdregarding the deprecation.config/approach.src/config.jsis still valid for older projects.📚 Documentation preview 📚: https://volto--7756.org.readthedocs.build/