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

Change focusRef prop-type to any for consent banner and brand #4382

Merged
merged 4 commits into from
Feb 26, 2021

Conversation

HarryVerhoef
Copy link
Contributor

Contributes to completion of bbc/simorgh#8855

Overall change: Change focusRef prop-type to fix HTMLAnchorElement is not defined error in Simorgh, for both psammead-brand and psammead-consent-banner.

Code changes:

  • Change focusRef prop-type to any.

  • 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

@HarryVerhoef HarryVerhoef changed the title Change focusRef prop-type to any Change focusRef prop-type to any for consent banner and brand Feb 26, 2021
@HarryVerhoef HarryVerhoef self-assigned this Feb 26, 2021
@HarryVerhoef HarryVerhoef marked this pull request as ready for review February 26, 2021 10:59
@HarryVerhoef HarryVerhoef added the ws-articles Tasks for the WS Articles Team label Feb 26, 2021
@github-actions
Copy link

Checkout your storybook preview here http://psammead-preview.tools.bbc.co.uk/4382

Copy link
Contributor

@andrewscfc andrewscfc left a comment

Choose a reason for hiding this comment

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

I've popped some comments and the brand component, I'd say you should do the same for consent banner

packages/components/psammead-brand/package.json Outdated Show resolved Hide resolved
packages/components/psammead-brand/src/index.jsx Outdated Show resolved Hide resolved
@HarryVerhoef HarryVerhoef merged commit cf2c01d into latest Feb 26, 2021
@HarryVerhoef HarryVerhoef deleted the change-reference-prop-type branch February 26, 2021 11:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
ws-articles Tasks for the WS Articles Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants