bedrock icon indicating copy to clipboard operation
bedrock copied to clipboard
verified publisher icon mozilla

Contentful split-based homepage

Open maureenlholland opened this issue 3 years ago • 2 comments

There are three steps involved

  1. Expand [Currently in progress on sandbox-2022-06-23 Contentful environment]
  • Add required split fields
  • Recreate homepage with split instead of hero
  1. Update
  • Update master alias to use new environment
  • Preview split homepage from master environment
  • Update Connect: Homepage to use new split homepage
  • Verify live homepage uses split homepage with no regression
  1. Contract
  • Delete hero entries
  • Delete hero content type
  • Remove hero-related code in Bedrock API

maureenlholland avatar Jun 28 '22 11:06 maureenlholland

@pmac r? for Step 1 https://app.contentful.com/spaces/w5er3c7zdgmd/environments/sandbox/entries/1Zq7Vu86z54mHCGTgHA6QN

Changes:

  • Add Mobile Media After field to Split Content Model (Because we can embed Logos/Wordmarks and CTAs in the rich text editor, I ended up not adding most of the fields from https://github.com/mozilla/bedrock/pull/11712/files#diff-77ac837e3992065e29560e9150f097cdea55bf59219a4d22ef08cb00f1427f5fR771, and will open a PR to remove this code)
  • Delete unpublished test split component
  • Fix any Mobile Display fields using the removed "Center conent" option

To test: Run locally with sandbox environment http://localhost:8000/en-US should still be the existing https://www.mozilla.org/en-US/ homepage (with hero components) http://localhost:8000/en-US/contentful-preview/1Zq7Vu86z54mHCGTgHA6QN/ should be the split-based home page

If all looks good, you can create a new environment V1.1 based off of this environment, and change the master alias. If something's off, ping me on this issue. Thanks!

maureenlholland avatar Jun 28 '22 13:06 maureenlholland

Actually, hold on a bit on the review here, might as well cover the MDN Plus content model updates too: https://github.com/mozilla/bedrock/issues/11356#issuecomment-1081687365

Edit: this is done in same sandbox. Ready for review including following changes and testing.

Changes

  • Add MDN Plus Wordmark and CTA
  • Add MDN Plus as Hero Product option
  • Add MDN Plus split section to split-based homepage
  • Update MDN Plus Product and CTA in hero-base homepage (still in draft)

To test on main branch http://localhost:8000/en-US unchanged http://localhost:8000/en-US/contentful-preview/1Zq7Vu86z54mHCGTgHA6QN/ has MDN plus section at bottom but wordmark is not available. Text centers and stacks on top in mobile view. on PR branch http://localhost:8000/en-US unchanged http://localhost:8000/en-US/contentful-preview/1Zq7Vu86z54mHCGTgHA6QN/ has MDN plus section & wordmark

maureenlholland avatar Jun 28 '22 14:06 maureenlholland

Split-based Contentful homepage is in production. Remaining Contentful hero code removed with https://github.com/mozilla/bedrock/pull/11687

maureenlholland avatar Feb 20 '23 10:02 maureenlholland