-
Notifications
You must be signed in to change notification settings - Fork 72
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
Add generic page around wrapper client libraries #191
Comments
|
@davidjgonzalez WDYT about the categories, especially the one to use for the page creation wizard? |
@kwin are you thinking a page documenting clientlibs categories in general and which can be used to hook into various aspects of the AEM UI? Why did you stop at the 4 use-cases you listed, vs things like DAM UI, or other aspects of AEM UI? FWIW - the categories you list look right (and dont recall the page wizard CL category off the top of my head).. |
IMHO it makes sense to document those categories centrally instead of per feature (especially as some JS might make sense in multiple use cases). As I don’t know of any other documentation page listing the categories it would make sense to maintain that at acs-aem-commons. The list is not complete but we can extend it once such a page is in place. |
@kwin sure - I think that would be good documentation to exist. In terms of using a feature, I'd rather just have the snippet that tells me exactly where it needs to go. I think asking users of features to try to understand where the right place to hook into is not good though; basically I think both could live in parallel, and the consolidated list would be more for feature-contributors to hook understand where to hook in. |
The only mention of wrapper client libraries (required due to Adobe-Consulting-Services/acs-aem-commons#1614) are at https://adobe-consulting-services.github.io/acs-aem-commons/pages/releases/4-0-0.html (where most links do no longer work). There should be a generic page (for all version >= 4.0) which lists
The text was updated successfully, but these errors were encountered: