Remove all Contentful-related code from the project#15845
Remove all Contentful-related code from the project#15845stevejalim wants to merge 3 commits intomainfrom
Conversation
|
@alexgibson Over to you for a scan over. If you're happy let's push to to the integration tests branch to get an early heads-up on any issues. Hopefully we'll have the VPN RC pages all published in all locales late next week, then we can merge this |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #15845 +/- ##
==========================================
+ Coverage 79.28% 79.70% +0.41%
==========================================
Files 159 152 -7
Lines 8343 7436 -907
==========================================
- Hits 6615 5927 -688
+ Misses 1728 1509 -219 ☔ View full report in Codecov by Sentry. |
* Drops the contentful app: views, models and templates * Drops Contentful-related settings * Drop static bundles that appeared to be for Contentful components only * Updates DB export script to not dump the contentful table * Does NOT do anything to automatically drop the contentful_ DB tables, however * Remove use of prefer_cms decorator for VPN RC views and moves the routing to cms_only_urls so that we can still reverse those page addresses * Move legacy /more/ redirects related to the VPN pages to global redirects (losing the locale-availability check, but deferring it to the CMS to 404 or not instead)
stevejalim
force-pushed
the
15843-remove-contentful
branch
from
2ca808c to
2d4fe74
Compare
Co-authored-by: Alex Gibson <alexgibson@users.noreply.github.com>
|
@stevejalim What's the status of this? It'd be good to merge the simplification. Also, it needs a rebase ;) |
|
Note: this is blocked on deciding what to do with the VPN pages with translations that existed in Contenful but do not yet have translations in Wagtail. |
|
Closed in favour of the fresher #16960 |
4 participants
DO NOT MERGE UNTIL THE VPN RC PAGES HAVE BEEN PUBLISHED IN ALL 10 LANGUAGES FROM THE NEW CMS. See #15846
This changeset removes Contentful-related code from our codebase.
Issue / Bugzilla link
Resolves #15843
Depends on #15846