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

Mushop contact page to accept Send a Note feedback input and store the data in ADB #284

Open
karthiksuku opened this issue Jul 16, 2021 · 8 comments
Labels
enhancement New feature or request

Comments

@karthiksuku
Copy link

Mushop Contact page to accept "Send a Note" feedback input and store the data in ADB

@jeevanjoseph
Copy link
Member

jeevanjoseph commented Jul 16, 2021

This is a good suggestion. However can you provide more info ?

There are multiple services that interact with ADB today. Orders uses ATP (shows typical JPA driven usage with hibernate under the covers), carts uses the JSON collection storage in ATP( shows using SODA), catalogue uses it for product info(shows Golang integration ), users service uses it too( shows nodeJS integration, and the node driver with an ORM ). Generally I’d say that every feature in the demo app needs to have a well defined purpose. Users reading the code should be able to gain something and a particular topic is ideally covered once and not redundantly.

Is there a specific DB feature, integration or practice that we can showcase with this enhancement?

@karthiksuku
Copy link
Author

Hi Jeevan , currently I have used used Order in ADB to trigger intergation to backend EBS / Fusion ERP using OIC.

Regarding the specifics with this reqest - What I am wanting is a way to capture feedback from the comments page - and it gets stored in ADB . Once its there - I am tyring to create APEX app and enbedd AI Language service to analyse the sentiment and display customer sentiment analysis APEX dashboard.

@jeevanjoseph
Copy link
Member

That’s an interesting one. Not sure why a DB or apex is required though. The Language service is seemingly is available over REST, and one could simply call the endpoint with text to analyze it, and display results right on the UI. If at all, perhaps an ephemeral store/cache like redis might suffice.

If you wanted to build a standalone apex app to showcase language service, you could simply build a form for text entry in apex itself.

The way I see this making sense for MuShop is to build a new service for sentiment analysis and use that service to call language service. However the credentials management needs serious thought.

@karthiksuku
Copy link
Author

karthiksuku commented Jul 17, 2021

well summarised re sentiment analysis as there are several ways to do this. At this stage the reqest is simple - Mushop to enable rudumentaty function to capture feedback @ Mushop Contact page and store in ADB.

@jeevanjoseph
Copy link
Member

The rudimentary persistence of feedback items to an ADB instance does not add anything new to MuShop, as we have multiple examples of persisting data into ADB with various languages and frameworks. Of Course, you're free to implement it for your needs if you choose. I don't think I'd favor a PR on that to the main branch at this time. @mvandervliet & @junior - what's your opinion ?

Sentiment analysis as a separate feature is pretty interesting and we can track that using a new issue.

@karthiksuku
Copy link
Author

Hi Jeevan , lets zoom to discuss. To use the sentiment analysis - we need to persist the feedback data somewhere... if it's in ADB it's easier for users to consume and innovate

@mvandervliet
Copy link
Contributor

Yes, warrants discussion. Generally speaking eCommerce apps always include features for reviews/ratings/feedback. In MuShop, this is omitted for sake of feature simplification where the concept is deliberately broad for the purpose of demonstration. I'm not opposed to accommodating this feature/service(s) with the assumption that administrative/inspection would be done via extensions (like apex) as a separate showcase. Any such demos or integrations should be also be documented and shareable.

@karthiksuku
Copy link
Author

Thank Matt / Jeevan

@junior junior added the enhancement New feature or request label Jul 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants