search

Adobe-Marketing-Cloud / aem-sample-we-retail

🚫 We.Retail has been archived, see instead the WKND Guide:
https://github.com/adobe/aem-guides-wknd
Apache License 2.0
123 stars 141 forks source link

Issue -690 Feature to take author to content fragment edit options fr… #164

Closed surya057 closed 5 years ago

surya057 commented 5 years ago

…om template level.

surya057 commented 5 years ago

Hi, This seems like a dependency issue. Can someone point me the right direction, please?

In my local machine, this build is a success. :|

surya057 commented 5 years ago

Hi Team, Can someone review and approve this PR, please?.

msagolj commented 5 years ago

@surya057 Are you aware that there is already a button for editing the content fragment in the toolbar ?

image

There is none for the content fragment list (not sure it makes much sense to add one there). If that's what you intend to do it should be added in the core components project, not here in the example project. This example project is showing of features of AEM that exist in the product.

surya057 commented 5 years ago

Thank you, For pointing it out. It is for content fragment list component. We are using plenty of content fragment list component so it is difficult to search for the proper content fragment group under content/dam path.

Sure, will add it to core components.

msagolj commented 5 years ago

@surya057 ok , thank you for the update. So i will close this one an you can create a new PR in the core components project