Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Export IconListContent and IconListIcon #2210

Closed
wants to merge 1 commit into from

Conversation

sawyerh
Copy link
Contributor

@sawyerh sawyerh commented Oct 27, 2022

Summary

The IconListContent and IconListIcon components weren't available to import from "@trussworks/react-uswds", but they appear to be required in order to use IconList.

How To Test

First time contributing! Not super sure how to test this one in particular, but happy to add one to catch regressions if folks have tips.

@sawyerh sawyerh requested a review from a user October 27, 2022 00:21
@sawyerh sawyerh changed the title Export IconListContent and IconListIcon fix: Export IconListContent and IconListIcon Oct 27, 2022
@jcbcapps
Copy link
Contributor

jcbcapps commented Nov 9, 2022

@sawyerh Thanks for contributing! Following our docs on contributions I've created a new PR here so that our build process can run, and will close this PR.

@jcbcapps jcbcapps closed this Nov 9, 2022
@sawyerh sawyerh deleted the fix-IconList-exports branch November 23, 2022 21:51
@lpsinger
Copy link
Contributor

When will this be fixed in a release?

@gidjin gidjin mentioned this pull request Feb 17, 2023
gidjin added a commit that referenced this pull request Feb 17, 2023
## [4.1.0](4.0.0...4.1.0) (2023-02-17)

### Features

- Make search component extendable ([#2230](#2230)) ([9bd9137](9bd9137))

### Bug Fixes

- Export IconListContent and IconListIcon ([#2210](#2210)) ([#2217](#2217)) ([5cb4024](5cb4024))
@gidjin
Copy link
Contributor

gidjin commented Feb 17, 2023

When will this be fixed in a release?

@lpsinger Good question, sorry I missed packaging a release in January like I had intended. But we just released 4.1.0 that includes this fix.

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.

5 participants