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[dev]: ✨ Update Celum Icon URLs to Avoid URL Blocking Issues #1212

Closed
4 of 9 tasks
yoezlem opened this issue Jul 17, 2024 · 0 comments · Fixed by #1281
Closed
4 of 9 tasks

feat[dev]: ✨ Update Celum Icon URLs to Avoid URL Blocking Issues #1212

yoezlem opened this issue Jul 17, 2024 · 0 comments · Fixed by #1281
Assignees
Labels
🔧 code needs changes in code

Comments

@yoezlem
Copy link
Contributor

yoezlem commented Jul 17, 2024

User Story

As a frontend developer of the Solid Design System, I would like to update the URLs for Celum icons from azureedge.net to union-investment.de, as well as ensure consistency across all repositories, so that we avoid issues with URL blocking in some Volksbanken, and maintain a reliable and unified access to our icons.

Suggested Solution

Replace all instances of fe-celum-icons-.azureedge.net with the new custom domain URLs in the frontend repositories.
Conduct a thorough review and testing to ensure that all icons are correctly accessible via the new URLs.

Technical Information

Azure DNS to New Custom Domain Mapping:
https://fe-celum-icons-dev.azureedge.net -> https://celum-icons.dev.fe.union-investment.de
https://fe-celum-icons-test.azureedge.net -> https://celum-icons.test.fe.union-investment.de
https://fe-celum-icons-prod.azureedge.net -> https://celum-icons.fe.union-investment.de

DoR

  • Item has business value
  • Item has been estimated by the team
  • Item is clear and well-defined
  • Item dependencies have been identified

DoD

  • Documentation has been created/updated (if applicable)
  • Migration Guide has been created/updated (if applicable)
  • Relevant E2E tests (Features, A11y, Bug fixes) are created/updated
  • Relevant stories (Features, A11y) are created/updated
  • Implementation works successfully on feature branch
@yoezlem yoezlem added the 🔧 code needs changes in code label Jul 17, 2024
@christophsaile christophsaile self-assigned this Aug 13, 2024
@christophsaile christophsaile linked a pull request Aug 13, 2024 that will close this issue
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔧 code needs changes in code
Projects
Status: ✅ Done
Development

Successfully merging a pull request may close this issue.

2 participants