Skip to content
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

Resource registry updates #1803

Closed
wants to merge 2 commits into from
Closed

Resource registry updates #1803

wants to merge 2 commits into from

Conversation

thet
Copy link
Member

@thet thet commented Dec 4, 2024

Document resource registry depends-all feature.

Ref: plone/Products.CMFPlone#4054
Ref: plone/Products.CMFPlone#4074

Add some clarifications on the resource registry purpose and what the depends setting is doing.


📚 Documentation preview 📚: https://plone6--1803.org.readthedocs.build/

@thet thet requested a review from stevepiercy December 4, 2024 11:13
@thet
Copy link
Member Author

thet commented Dec 4, 2024

@stevepiercy I‌ just recognized you also wrote docs for this feature on #1802
thanks for that!
I think this PR is still a good addition to yours.

@stevepiercy
Copy link
Contributor

@thet I'll move content from this PR into that one, so we don't have to deal with merge conflicts. Thank you!

stevepiercy added a commit that referenced this pull request Dec 4, 2024
@stevepiercy
Copy link
Contributor

Content merged in #1802. Closing this one.

@stevepiercy stevepiercy closed this Dec 4, 2024
stevepiercy added a commit that referenced this pull request Dec 6, 2024
#1802)

* Document how to specify a static resource bundle to load as the last one in Classic UI

* Merge content from #1803

* Add clause for multiple `all`s

* More merge and removal of redundant content

* Update docs/classic-ui/static-resources.md

Co-authored-by: Peter Mathis <[email protected]>

* Update docs/classic-ui/static-resources.md

Co-authored-by: Yuri <[email protected]>

* Retain friendly "you", but use "both"

---------

Co-authored-by: Peter Mathis <[email protected]>
Co-authored-by: Johannes Raggam <[email protected]>
Co-authored-by: Yuri <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants