-
-
Notifications
You must be signed in to change notification settings - Fork 183
add deprecation chapter #1850
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: 6.0
Are you sure you want to change the base?
add deprecation chapter #1850
Conversation
Co-authored-by: Alessandro Pisa <[email protected]>
Co-authored-by: Alessandro Pisa <[email protected]>
Co-authored-by: Alessandro Pisa <[email protected]>
Co-authored-by: Alessandro Pisa <[email protected]>
Co-authored-by: Alessandro Pisa <[email protected]>
Co-authored-by: Alessandro Pisa <[email protected]>
Co-authored-by: Alessandro Pisa <[email protected]>
Co-authored-by: Alessandro Pisa <[email protected]>
Co-authored-by: Alessandro Pisa <[email protected]>
Co-authored-by: Alessandro Pisa <[email protected]>
Co-authored-by: Alessandro Pisa <[email protected]>
Co-authored-by: Alessandro Pisa <[email protected]>
Co-authored-by: Alessandro Pisa <[email protected]>
Co-authored-by: Alessandro Pisa <[email protected]>
Co-authored-by: Alessandro Pisa <[email protected]> Co-authored-by: Jens W. Klein <[email protected]>
docs/backend/index.md
Outdated
| configuration-registry | ||
| content-types/index | ||
| control-panels | ||
| deprecation |
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.
Do you all think that this insertion point in the TOC is the most appropiate place for this subject?
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.
It is not. See #1850 (review).
|
@MrTango I'm awaiting changes per #1850 (review) before merging. Please let me know. Thank you! |
|
@MrTango should I close this PR as abandoned and won't fix? It needs help before it can be merged. |
It would be a pity because it is a very nice write up :/ |
Indeed. However, my review has not been addressed. I don't know why. No response from @MrTango implies this PR is not needed. If you care to push it forward, please be my guest. |
I will take a look. I wrote this initially back in the days... so I would like to see it survive. |
2ccbcf2 to
46b14eb
Compare
|
Ok, this should work. @stevepiercy what do you think? |
Co-authored-by: Alessandro Pisa <[email protected]>
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.
Thank you very much to all of you :)
@stevepiercy do you mind having another look?
- Generalize for npm and Volto - Clean up MyST style and formatting
First-time contributors
You must read and follow our First-time contributors.
Submit a pull request
Thank you for your contribution to the Plone Documentation.
Before submitting this pull request, please make sure you follow our guides:
Issue number
Description
Write a description of the fixes or improvements.
Add screenshots or links to a preview of the changes
📚 Documentation preview 📚: https://plone6--1850.org.readthedocs.build/backend/deprecation.html