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: add custom translations for webchat UI components #68

Merged
merged 4 commits into from
Mar 7, 2025

Conversation

vj-venkatesan
Copy link
Contributor

@vj-venkatesan vj-venkatesan commented Feb 26, 2025

Success criteria

Please describe what should be possible after this change. List all individual items on a separate line.
See (Ticket)

  • Custom translations text configured in endpoint editor should be used for
    • Disconnect overlay text
    • Home screen heading used for screen readers

How to test

Please describe the individual steps on how a peer can test your change.

  1. Configure webchat3 endpoint with custom translations
  2. Open webchat , start a new conversation
  3. Disconnect from network and observe customTranslation string from customJson is used

Security

  • Possible injection vector
  • Authentication/Access controls touched
  • Sensitive Data could be exposed
  • XSS
  • Logging/Monitoring touched
  • Exchanges data with external systems
  • No security implications

Additional considerations

  • This PR might have performance implications

Documentation Considerations

These are hints for the documentation team to help write the docs.

@vj-venkatesan vj-venkatesan marked this pull request as ready for review February 26, 2025 15:42
sushmi21
sushmi21 previously approved these changes Mar 5, 2025
@vj-venkatesan vj-venkatesan force-pushed the feature/#91449-text-translation-customization branch from ffe6856 to 0fc2000 Compare March 5, 2025 13:13
sushmi21
sushmi21 previously approved these changes Mar 6, 2025
@vj-venkatesan vj-venkatesan merged commit 9a4e0eb into main Mar 7, 2025
5 of 6 checks passed
@sushmi21 sushmi21 mentioned this pull request Mar 7, 2025
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants