jaketracey / opax

Opax is an integration tool that brings the capabilities of OpenAI's ChatGPT into Adobe Experience Manager (AEM). It leverages AEM's robust content management features and combines them with the versatility and power of ChatGPT, allowing for a sophisticated, AI-driven user experience.
https://opax.ai
MIT License
6 stars 7 forks source link

Renderer aemgpt/components/page is missing in the code repository. #4

Closed shivakumar660 closed 1 year ago

shivakumar660 commented 1 year ago

I have followed all the steps, after installing the package, when I try to hit http://localhost:4502/content/aemgpt/us/en.html I get the following error message. I noticed that /apps/aemgpt has only clientlibs and osgiconfig, where are the components ? where is the rendered for "aemgpt/components/page" defined ?

02.07.2023 22:14:31.624 ERROR [[0:0:0:0:0:0:0:1] [1688316271613] GET /content/aemgpt/us/en.html HTTP/1.1] org.apache.sling.servlets.get.impl.DefaultGetServlet No renderer for extension html, cannot render resource JcrNodeResource, type=aemgpt/components/page, superType=null, path=/content/aemgpt/us/en/jcr:content

jaketracey commented 1 year ago

Hi @shivakumar660

Should work now. As the project doesn't need the archetype components I removed them. Will add some additional examples in future!

Let me know if that works for you.

shivakumar660 commented 1 year ago

Hi Jake, Thank you for the quick response. I see that in the filter.xml of ui.apps, you have added the path till clientlibs "//apps/aemgpt/clientlibs" , so its not picking the components as part of deployment. ct? I would like to contribute for bug fixes or new features development. I have around 10 years of experience as a AEM Developer , here is my linkedIn - https://www.linkedin.com/in/shivakumar-chandrappa-3221642a/

jaketracey commented 1 year ago

Hi Shivakumar,

Feel free to make a PR! Happy to have any contributions.

The integration itself does not need any components added, so unless for examples there really isn't a need for any to be included in this package.

The author clientlib attaches to the components on this line: https://github.com/jaketracey/opax/blob/7a7c50d5fc78b8c957b763272610979bda0ab80b/ui.frontend/src/main/webpack/site/main.js#L22C27-L22C40

Ultimately we want to modify this so that it uses the OSGI configuration so that it's configurable which component the integration attaches to (not just wcm/text/v2 as it currently is.

shivakumar660 commented 1 year ago
image

I am getting this message when I try to raise a PR "Pull request creation failed. Validation failed: must be a collaborator"

jaketracey commented 1 year ago

Fixed - should work now!

shivakumar660 commented 1 year ago

https://github.com/jaketracey/opax/pull/7 I have created a PR.

shivakumar660 commented 1 year ago

PR is merged. Hence closing this issue.