Skip to content
This repository has been archived by the owner on Aug 13, 2023. It is now read-only.

Migrate [level 2] packages to Emotion #3891

Merged
merged 52 commits into from
Oct 16, 2020
Merged

Conversation

simonsinclair
Copy link
Contributor

@simonsinclair simonsinclair commented Oct 15, 2020

Relates to #3763.

Overall change:

  • psammead-timestamp
  • psammead-sitewide-links
  • psammead-brand
  • psammead-byline
  • psammead-grid
  • psammead-story-promo
  • psammead-social-embed
  • psammead-paragraph
  • psammead-headings
  • psammead-useful-links
  • psammead-script-link
  • psammead-image-placeholder
  • psammead-section-label
  • psammead-bulleted-list
  • psammead-heading-index
  • psammead-consent-banner
  • psammead-copyright

Code changes:


  • I have assigned myself to this PR and the corresponding issues
  • Automated jest tests added (for new features) or updated (for existing features)
  • This PR requires manual testing

@simonsinclair simonsinclair added technical-work Technical debt, support work and building new technical tools and features cross-team For visibility for both World Service teams (Engage & Media) labels Oct 15, 2020
@simonsinclair simonsinclair added this to the Migrate to Emotion milestone Oct 15, 2020
@simonsinclair simonsinclair self-assigned this Oct 15, 2020
Copy link
Contributor

@ryanmccombe ryanmccombe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Still need to add PR numbers to changelog

const paddingDir = ({ dir }) => `padding-${dir === 'rtl' ? 'left' : 'right'}`;

const TextGridColumnsTopStory = css`
const TextGridColumnsTopStory = `
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did we do a visual comparison on storybook between this branch and the storybook on github? Lot of changes here 👍

packages/components/psammead-brand/CHANGELOG.md Outdated Show resolved Hide resolved
packages/components/psammead-byline/CHANGELOG.md Outdated Show resolved Hide resolved
packages/components/psammead-grid/CHANGELOG.md Outdated Show resolved Hide resolved
packages/components/psammead-sitewide-links/CHANGELOG.md Outdated Show resolved Hide resolved
packages/components/psammead-story-promo/CHANGELOG.md Outdated Show resolved Hide resolved
packages/components/psammead-timestamp/CHANGELOG.md Outdated Show resolved Hide resolved
Copy link
Contributor

@RichardPK RichardPK left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Version bumps on changelogs and then good!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cross-team For visibility for both World Service teams (Engage & Media) technical-work Technical debt, support work and building new technical tools and features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants