bedrock icon indicating copy to clipboard operation
bedrock copied to clipboard

Contentful Clean-up

Open maureenlholland opened this issue 3 years ago • 1 comments

We want to de-clutter Contentful-related content and prioritize remaining issues

CONTENTFUL WEB APP

BEDROCK API

  • [ ] No more TODO in comments (or at least, all todos are linked to issues so we don't lose track of them)

OPEN ISSUES If still relevant and not already there, maybe add to marketing-platform project board?

Workflow

  • [ ] https://github.com/mozilla/bedrock/issues/9467
  • [ ] https://github.com/mozilla/bedrock/issues/9469
  • [ ] https://github.com/mozilla/bedrock/issues/9472
  • [ ] https://github.com/mozilla/bedrock/issues/9901
  • [ ] https://github.com/mozilla/bedrock/issues/9988
  • [ ] https://github.com/mozilla/bedrock/issues/9993
  • [ ] https://github.com/mozilla/bedrock/issues/9994
  • [ ] https://github.com/mozilla/bedrock/issues/9996
  • [ ] https://github.com/mozilla/bedrock/issues/9997

Enhancements

  • [ ] https://github.com/mozilla/bedrock/issues/10678
  • [ ] https://github.com/mozilla/bedrock/issues/10742
  • [ ] https://github.com/mozilla/bedrock/issues/10743
  • [x] https://github.com/mozilla/bedrock/issues/10773
  • [ ] https://github.com/mozilla/bedrock/issues/10785
  • [ ] https://github.com/mozilla/bedrock/issues/10898
  • [ ] https://github.com/mozilla/bedrock/issues/10899
  • [ ] https://github.com/mozilla/bedrock/issues/10443
  • [ ] https://github.com/mozilla/bedrock/issues/9992
  • [ ] https://github.com/mozilla/bedrock/issues/12027

Bugs [mostly related to homepage, should be fixed with split-based homepage update coming soon]

  • [ ] https://github.com/mozilla/bedrock/issues/10667
  • [ ] https://github.com/mozilla/bedrock/issues/10668
  • [ ] https://github.com/mozilla/bedrock/issues/10677
  • [ ] https://github.com/mozilla/bedrock/issues/10765

Design system checklist

  • [ ] https://github.com/mozilla/bedrock/issues/10127 remaining unchecked
  • rich text image support
  • high/low res image solutions for split, hero, cards, and rich text
  • include CSS as needed for nested components (i.e. youtube videos in cards)
  • include JS as needed for nested components (i.e. youtube videos in cards)
  • figure out section/background wrappers
  • add split support inside rich text areas
  • filter improvements: download button shouldn't be wrapped in p tags, safe guard against dev/staging links
  • CTA button for Monitor, tracking params
  • CTA app store links for Firefox mobile and Pocket
  • l10n: translated feature pages, DE homepage
  • analytics: identify CTAs by position on page
  • include Pocket feed on homepage
  • add all Protocol icons to media library
  • add all product OG images to media library
  • docs for Contentful available components in Protocol
  • center logo or wordmark when text is centered
  • picto: request icon without aspect ratio
  • section heading: zap support
  • card layout: "ragged" image aspect ratios
  • multi-column: centered text option

maureenlholland avatar Jun 28 '22 11:06 maureenlholland

Going forward, I'd put the top 2 priorities as:

  1. ~l10n integration (Smartling)~ done!
  2. Ship-it Workflow (order of operations when updates happen across Contentful API and bedrock code, while editors might be changing content in Contentful master environment), see https://github.com/mozilla/bedrock/issues/12319

(*3. ~Change to new Compose model~ done!)

maureenlholland avatar Jun 30 '22 17:06 maureenlholland

@stephaniehobson I tried to round up our outstanding Contentful issues for re-evaluation. Take a look when you have time to see if there are any we can close or should prioritize (i.e. responsive images)

maureenlholland avatar Jan 13 '23 10:01 maureenlholland

closing in favor of more specific issues and updated label organization (see Contentful labelled issues)

maureenlholland avatar Feb 28 '23 18:02 maureenlholland