Skip to content

Conversation

@dmatis
Copy link
Contributor

@dmatis dmatis commented May 12, 2020

Description

Fixes an issue where the Username is cut off from Shellbar (#1002)

  • Popover and Shellbar stories were missing an export in order to appear in the docs
  • Removed a duplicate story in Shellbar (primary)
  • ProductMenu was missing an array and causing errors in Storybook
  • Disabled Popover storyshot containing React hook

@dmatis dmatis requested a review from jbadan May 12, 2020 21:31
@CLAassistant
Copy link

CLAassistant commented May 12, 2020

CLA assistant check
All committers have signed the CLA.

@netlify
Copy link

netlify bot commented May 12, 2020

Deploy preview for fundamental-react ready!

Built with commit 8bbbb35

https://deploy-preview-1008--fundamental-react.netlify.app

@dmatis dmatis changed the title fix: Add popper and shellbar to storybook chore: Add popper and shellbar to storybook May 12, 2020
Copy link
Contributor

@jbadan jbadan left a comment

Choose a reason for hiding this comment

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

Can you remove all the _playground files that are accidentally getting checked in?

you'll also need to run npm run test:storybook:update
https://github.com/SAP/fundamental-react/wiki/Testing#npm-run-teststorybookupdate

@dmatis dmatis changed the title chore: Add popper and shellbar to storybook fix: Username is cut off from Shellbar May 12, 2020
Copy link
Contributor

@jbadan jbadan left a comment

Choose a reason for hiding this comment

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

once travis passes 🚢

@dmatis dmatis merged commit 8d36439 into master May 15, 2020
@dmatis dmatis deleted the fix-missing-storybook-exports branch May 15, 2020 20:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants