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

feat: Intershop Copilot integration #1698

Open
wants to merge 10 commits into
base: develop
Choose a base branch
from
Open

Conversation

714Lemons
Copy link

@714Lemons 714Lemons commented Jul 15, 2024

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

@SGrueber SGrueber marked this pull request as draft July 15, 2024 10:54
@714Lemons 714Lemons self-assigned this Jul 31, 2024
@shauke shauke added this to the 5.3 milestone Jan 28, 2025
@shauke shauke self-assigned this Jan 28, 2025
@shauke shauke self-requested a review February 3, 2025 09:56
@shauke shauke changed the title SHOWCASE - Copilot PoC feat: Intershop Copilot integration Feb 3, 2025
714Lemons and others added 2 commits February 17, 2025 16:50
- 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
@shauke shauke requested review from a team and SGrueber February 17, 2025 20:56
@shauke shauke added the feature New feature or request label Feb 17, 2025
@shauke shauke marked this pull request as ready for review February 17, 2025 20:57
Copy link

Azure Demo Servers are available:

@shauke shauke requested a review from M-Behr February 18, 2025 10:12
Copy link

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>.",
Copy link
Contributor

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.

Copy link
Author

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)

Copy link
Contributor

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.

Copy link
Author

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.

mglatter
mglatter previously approved these changes Feb 18, 2025
Copy link
Contributor

@mglatter mglatter left a 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.

@mglatter mglatter self-requested a review February 18, 2025 16:15
mglatter
mglatter previously approved these changes Feb 19, 2025
Copy link

Azure Demo Servers are available:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants