Skip to content

fix: Export Banner component#752

Merged
engfragui merged 2 commits intomainfrom
francesca/export-banner
Jan 30, 2023
Merged

fix: Export Banner component#752
engfragui merged 2 commits intomainfrom
francesca/export-banner

Conversation

@engfragui
Copy link
Copy Markdown
Contributor

@engfragui engfragui commented Jan 30, 2023

Follows up to #750

Short description

In #750 I created a new Banner component, but I forgot to actually export it in the top-level index file, so it's not actually usable from outside Reactist. 😅

For the time being, I added a small note to the README so next time I won't forget (maybe), but I will see if plop can programmatically do this for us next time.

PR Checklist

  • Added tests for bugs / new features
  • Updated docs (storybooks, readme)
  • Executed npm run validate and made sure no errors / warnings were shown
  • Described changes in CHANGELOG.md
  • Bumped version in package.json and package-lock.json (npm --no-git-tag-version version <major|minor|patch>) ref
  • Updated all static build artifacts (npm run build-all)

Versioning

Patch bump.

@engfragui engfragui self-assigned this Jan 30, 2023
@engfragui engfragui marked this pull request as ready for review January 30, 2023 14:06
@engfragui engfragui added the 👀 Show PR Used for PRs that need a review, but can be merged when CI is green. label Jan 30, 2023
@engfragui engfragui requested a review from gnapse January 30, 2023 14:08
@engfragui engfragui merged commit aad0405 into main Jan 30, 2023
@engfragui engfragui deleted the francesca/export-banner branch January 30, 2023 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

👀 Show PR Used for PRs that need a review, but can be merged when CI is green.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant