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
DOCS add note to changelog for Solr changes #9576
DOCS add note to changelog for Solr changes #9576
Conversation
I know we haven't quite landed on an approach for capturing information of non-core modules in changelogs, but I think this one is really worth including. Addons has the fulltextsearch module downloads at 158,435 and given the changes, I think it's better to be safe than sorry?
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. I'm not sure about putting satellite module changes in the core changelogs either, so I'll wait for someone else to make that determination.
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.
Agree. I think at this point we just don't have a better place than the changelog to mention such impactful things.
Even though technically fulltext-search module is not a part of the CMS Recipes, I believe having that note in the changelog will do no harm.
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.
One minor class typo, otherwise happy for this to go ahead (echoing @dnsl48's thoughts on the matter.)
Co-authored-by: Garion Herman <garion@silverstripe.com>
I know we haven't quite landed on an approach for capturing information of non-core modules in changelogs, but I think this one is really worth including. Addons has the fulltextsearch module downloads at 158,435 and given the changes, I think it's better to be safe than sorry?