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

Adds Brand component to Psammead #105

Merged
merged 31 commits into from
Dec 11, 2018
Merged

Adds Brand component to Psammead #105

merged 31 commits into from
Dec 11, 2018

Conversation

radiocontrolled
Copy link
Contributor

@radiocontrolled radiocontrolled commented Dec 4, 2018

Resolves #56

Adds a Brand component to Psammead

screenshot
screen shot 2018-12-07 at 17 00 00

  • I have assigned myself to this PR and the corresponding issue
  • [N/A] Tests added for new features
  • Test engineer approval

@radiocontrolled
Copy link
Contributor Author

Currently having an issue using styled-components' styled.css and @sareh and I have raised an issue in: styled-components/babel-plugin-styled-components#188

@mxstbr
Copy link

mxstbr commented Dec 7, 2018

By the way, does this mean BBC News is building their internal component library with styled-components in the open? Because that would be incredibly exciting! 🎉

@radiocontrolled
Copy link
Contributor Author

Actually the above error I comment on was because I needed to do

import styled, { css } from 'styled-components'; in src/index.jsx`

.stylelintrc Outdated Show resolved Hide resolved
Copy link
Contributor

@ChrisBAshton ChrisBAshton left a comment

Choose a reason for hiding this comment

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

Thanks Alison 👍

.stylelintrc Outdated Show resolved Hide resolved
babel.config.js Show resolved Hide resolved
packages/components/psammead-brand/README.md Outdated Show resolved Hide resolved
@radiocontrolled
Copy link
Contributor Author

radiocontrolled commented Dec 7, 2018

Your comments are addressed @ChrisBAshton

@radiocontrolled radiocontrolled dismissed stale reviews from sareh via f7ba1b5 December 10, 2018 10:48
bcmn
bcmn previously approved these changes Dec 10, 2018
dr3
dr3 previously approved these changes Dec 10, 2018
Copy link
Contributor

@dr3 dr3 left a comment

Choose a reason for hiding this comment

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

@radiocontrolled radiocontrolled dismissed stale reviews from dr3 and bcmn via 219498d December 11, 2018 10:27
dr3
dr3 previously approved these changes Dec 11, 2018
ChrisBAshton
ChrisBAshton previously approved these changes Dec 11, 2018
@radiocontrolled radiocontrolled dismissed stale reviews from ChrisBAshton and dr3 via 0d414fa December 11, 2018 10:59
@jamesbrumpton
Copy link
Contributor

LGTM

@radiocontrolled radiocontrolled merged commit 3bd31c5 into latest Dec 11, 2018
@radiocontrolled radiocontrolled deleted the add-brand branch December 11, 2018 11:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants