plone / volto

React-based frontend for the Plone Content Management System
https://demo.plone.org/
MIT License
475 stars 643 forks source link

Add "profile" term to Glossary #3071

Closed stevepiercy closed 2 years ago

stevepiercy commented 2 years ago

@stevepiercy The add-ons install "profiles" are like the Plone Generic Setup ones. That's why we chose the naming like this.

Should we define the term "profile" in the Glossary? I think we should make it clear that a profile in Volto and a profile in GenericSetup are similar, but have some differences. I could create a new issue for that, so we can merge this one and move on.

Originally posted by @stevepiercy in https://github.com/plone/volto/issues/3067#issuecomment-1041286094

tiberiuichim commented 2 years ago

Maybe we should use "addon loader" and "additional addon loaders", to avoid the overlap with "profile"? @sneridagh

sneridagh commented 2 years ago

So, I'm ok with @tiberiuichim proposal and turns out... we already have one entry for this!

https://github.com/plone/documentation/blob/a18f60aa264b859c078eb3477ee9aa21849666ca/docs/glossary.md#L137

@stevepiercy it turns out you wrote it already, using the "add-on loader" naming. So I'm ok with this. Although I'll improve it a bit.

sneridagh commented 2 years ago

https://github.com/plone/documentation/pull/1185

stevepiercy commented 2 years ago

So, I'm ok with @tiberiuichim proposal and turns out... we already have one entry for this!

https://github.com/plone/documentation/blob/a18f60aa264b859c078eb3477ee9aa21849666ca/docs/glossary.md#L137

@stevepiercy it turns out you wrote it already, using the "add-on loader" naming. So I'm ok with this. Although I'll improve it a bit.

Heh, it goes deeper. The documentation glossary was a merger of all the various glossaries and that from training. The one from training was a conversion from rst to md in the training repo, and before that it was a consolidation of all the glossaries: https://github.com/plone/training/blame/e55aa6b51025270db54a34781dc93e4b3abc4142/voltoaddons/about/glossary.rst#L14-L15

stevepiercy commented 2 years ago

Closed by https://github.com/plone/documentation/pull/1185