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
[4.x] Fixed newsletter header image path #14690
[4.x] Fixed newsletter header image path #14690
Conversation
refs https://github.com/TryGhost/Team/issues/1579 - When writing to the database, the header_image is tranformed to the transformReady path - When reading from the database, the transformReady path is transformed to an absolute path - Includes a test when adding a newsletter
refs https://github.com/TryGhost/Team/issues/1579 - Updates all newsletter who have a header_image to make sure it is saved in transform ready format - Down operation is required to work with the old model logic and transforms it back to an absolute format
It looks like this PR contains a migration 👀 General requirements
Data changes
|
Codecov Report
@@ Coverage Diff @@
## 4.x #14690 +/- ##
==========================================
- Coverage 59.85% 59.83% -0.03%
==========================================
Files 580 580
Lines 47993 48016 +23
Branches 4217 4217
==========================================
+ Hits 28727 28730 +3
- Misses 19225 19245 +20
Partials 41 41
Continue to review full report at Codecov.
|
refs https://github.com/TryGhost/Team/issues/1579 - When writing to the database, the header_image is tranformed to the transformReady path - When reading from the database, the transformReady path is transformed to an absolute path - Includes a test when adding a newsletter Migration: - Updates all newsletter who have a header_image to make sure it is saved in transform ready format - Down operation is required to work with the old model logic and transforms it back to an absolute format
refs https://github.com/TryGhost/Team/issues/1579 - When writing to the database, the header_image is tranformed to the transformReady path - When reading from the database, the transformReady path is transformed to an absolute path - Includes a test when adding a newsletter Migration: - Updates all newsletter who have a header_image to make sure it is saved in transform ready format - Down operation is required to work with the old model logic and transforms it back to an absolute format
refs https://github.com/TryGhost/Team/issues/1579
Migration:
5.x version: #14688