Skip to content
This repository was archived by the owner on Jan 9, 2023. It is now read-only.

Update storybook docs for breadcrumbs, calendar and callout#637

Merged
matteovivona merged 7 commits intoHospitalRun:masterfrom
RobertoMSousa:breadcrumb_docs
Nov 8, 2020
Merged

Update storybook docs for breadcrumbs, calendar and callout#637
matteovivona merged 7 commits intoHospitalRun:masterfrom
RobertoMSousa:breadcrumb_docs

Conversation

@RobertoMSousa
Copy link
Copy Markdown
Contributor

@RobertoMSousa RobertoMSousa commented Oct 28, 2020

Fixes callout, breadcrumbs and calendar stories.

Changes proposed in this pull request:
Update the docs of the components mentioned previously to allow the storybook 6 to render the docs page and fix a bug on the calendar that was preventing the component to be rendered at all on storybook.

@RobertoMSousa RobertoMSousa changed the title Update sb Update storybook docs for breadcrumbs, calendar and callout Oct 28, 2020
Comment thread stories/callout.stories.tsx Outdated
Comment thread stories/calendar.stories.tsx Outdated
Copy link
Copy Markdown

@blestab blestab left a comment

Choose a reason for hiding this comment

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

LGTM if you can just address the changes suggested by @jackcmeyer

Copy link
Copy Markdown

@blestab blestab left a comment

Choose a reason for hiding this comment

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

LGTM if you can just address the changes suggested by @jackcmeyer

@RobertoMSousa
Copy link
Copy Markdown
Contributor Author

@blestab , typo fixed.
thanks 🙂

@RobertoMSousa RobertoMSousa requested a review from blestab November 7, 2020 18:18
@blestab
Copy link
Copy Markdown

blestab commented Nov 7, 2020

Thanks @RobertoMSousa
May you address the commented out import in stories/callout.stories.tsx as per @jackcmeyer comment i.e. if we are no longer using it then let's remove it.

@RobertoMSousa
Copy link
Copy Markdown
Contributor Author

RobertoMSousa commented Nov 8, 2020

Thanks @RobertoMSousa
May you address the commented out import in stories/callout.stories.tsx as per @jackcmeyer comment i.e. if we are no longer using it then let's remove it.

That was already removed on this commit

@matteovivona matteovivona self-requested a review November 8, 2020 13:19
@matteovivona matteovivona merged commit f06ef32 into HospitalRun:master Nov 8, 2020
@ghost
Copy link
Copy Markdown

ghost commented Nov 8, 2020

🎉 This PR is included in version 3.0.4 🎉

The release is available on:

Your semantic-release bot 📦🚀

@ghost ghost added the released label Nov 8, 2020
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.

4 participants