-
Notifications
You must be signed in to change notification settings - Fork 86
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
feat: Intershop Copilot integration #1698
base: develop
Are you sure you want to change the base?
Conversation
9eba747
to
0958699
Compare
0958699
to
13a0c50
Compare
86884dd
to
4337841
Compare
- created new copilot extension - included environment setup - added component to app.component.html
* add copilot configuration state and cleanup/restructure copilot store to common implementation style * no longer use injection key * make copilot component lazy component * restructured Copilot script loading * Copilot localizations * Copilot tool call cleanup * Copilot integration documentation
4337841
to
eb92fb0
Compare
Azure Demo Servers are available: |
Azure Demo Servers are available: |
@@ -893,6 +893,18 @@ | |||
"cookies.modal.accept_all": "Accept all", | |||
"cookies.modal.cancel": "Cancel", | |||
"cookies.modal.save": "Save settings", | |||
"copilot.disclaimer.buttonText": "Start Chatting", | |||
"copilot.disclaimer.message": "By using this chatbot, you agree to the <a target=\"_blank\" href=\"https://intershop.com/en/terms-of-use\">Terms & Conditions</a>.", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There doesn't seem to exist any German or French equivalent of that page, so we will need to refer to the English one for all locales for the time being.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
German: https://www.intershop.com/de/datenschutz
French: https://www.intershop.com/fr/protection-des-donnees
But the AI-paragraph seems to only be on the german and english site currently. (I cant speek french)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was about T&C, not Privacy Policy.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh sry i think this should be linked to Privacy Policy, i dont see any copilot related information on the Terms & Conditions.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Localization approved, documentation review still needs to be done.
Note: The Copilot part of the localization is not in sentence case as the remaining PWA localization, because the Copilot styling cannot be accessed, and thus, it should be written as it will be displayed.
Azure Demo Servers are available: |
PR Type
[x] Feature
What Is the New Behavior?
This is the integration of the Intershop Copilot which uses Flowise Embed to deliver an "intelligent assistant".
The used customized Intershop fork for the embedded chatbot can be found at https://github.com/intershop/ai-flowise-chat-embed.
Does this PR Introduce a Breaking Change?
[x] No
Other Information
AB#98164