-
Notifications
You must be signed in to change notification settings - Fork 67
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
Review our OSS contribution doc #360
Labels
Comments
This was referenced Oct 4, 2023
This was referenced Oct 9, 2023
@emteknetnz Looks like you've approved a lot of PRs but not merged anything? |
Assigned back to @GuySartorelli for |
This was referenced Oct 18, 2023
Reopening for the remaining todo stuff |
@GuySartorelli have merged the new PR - is there anything left? You can probably re-close? |
That's all this time |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
This issue used to be https://github.com/silverstripeltd/product-issues/issues/783 but there was no need for it to be private.
At Stripecon 2023 we're giving a talk that aims to promote contributions to Silverstripe CMS open source. We should make sure our documentation is up to date so that people can contribute confidently and effectively.
Acceptance criteria
Notes
PRs
IMPORTANT These are likely to have significant merge conflicts with CMS 5 docs. When you have merged all the PRs, reassign to Guy so they can merge up and resolve the conflicts.
Just content changes
Just updating content - not moving anything, ideally
Content and some structural changes
Updates to content, and moving content around between pages
Pure structural changes
Reordering pages altogether
The text was updated successfully, but these errors were encountered: