-
Notifications
You must be signed in to change notification settings - Fork 105
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
chore: [IOPID-1743] Change CDN urls #5719
Conversation
Affected stories
|
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.
LGTM for the messages' section
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.
LGTM
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #5719 +/- ##
==========================================
+ Coverage 48.42% 48.86% +0.43%
==========================================
Files 1488 1584 +96
Lines 31617 31872 +255
Branches 7669 7722 +53
==========================================
+ Hits 15311 15574 +263
+ Misses 16238 16230 -8
Partials 68 68
... and 123 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
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.
LGTM
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.
LGTM
🎉 All dependencies have been resolved ! |
Warning
This PR depends on pagopa/io-services-metadata#779 and pagopa/io-dev-api-server#371
Short description
Change CDN urls from assets.cdn.io.italia.it to assets.cdn.io.pagopa.it for all occurrence
How to test
Verify that each link works identically to the as-is