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

chore(Icon): adds test for when icon provided is FC using hooks #3699

Merged
merged 1 commit into from
Jun 13, 2024

Conversation

langz
Copy link
Contributor

@langz langz commented Jun 12, 2024

  • Added a test case

Copy link

vercel bot commented Jun 12, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
eufemia ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 13, 2024 10:38am

@langz langz changed the title fix(Icon): prevent breaking when icon is component with hook fix(Icon): prevent breaking when icon provided is FC using hooks Jun 12, 2024
@langz
Copy link
Contributor Author

langz commented Jun 13, 2024

For fun, I checked if this test worked in #3703, and it does, so that's great.

@langz langz closed this Jun 13, 2024
@langz langz reopened this Jun 13, 2024
@langz langz changed the title fix(Icon): prevent breaking when icon provided is FC using hooks chore(Icon): adds test for when icon provided is FC using hooks Jun 13, 2024
@langz langz force-pushed the fix/hooks-error-using-form-status-icons-in-icon branch from 937860f to d961d95 Compare June 13, 2024 08:46
@langz langz changed the base branch from main to feat/icon-typescript June 13, 2024 08:47
@langz langz force-pushed the fix/hooks-error-using-form-status-icons-in-icon branch 2 times, most recently from d2ced20 to c005ecd Compare June 13, 2024 08:47
@tujoworker tujoworker force-pushed the feat/icon-typescript branch from 1dfec8c to 01beba3 Compare June 13, 2024 09:57
Base automatically changed from feat/icon-typescript to main June 13, 2024 10:18
tujoworker added a commit that referenced this pull request Jun 13, 2024
Will solve the issue described in this PR #3699

Even all the tests and types are running green, it needs extensive
manual testing.

---------

Co-authored-by: Anders <anderslangseth@gmail.com>
@langz langz force-pushed the fix/hooks-error-using-form-status-icons-in-icon branch from c005ecd to ff5f10a Compare June 13, 2024 10:22
Copy link

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Copy link
Member

@tujoworker tujoworker left a comment

Choose a reason for hiding this comment

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

Whoop 🙌

@langz langz merged commit bb4bccc into main Jun 13, 2024
10 checks passed
@langz langz deleted the fix/hooks-error-using-form-status-icons-in-icon branch June 13, 2024 15:10
tujoworker pushed a commit that referenced this pull request Jun 14, 2024
## [10.36.0](v10.35.0...v10.36.0) (2024-06-14)

### ✨ Features

* **Anchor:** add helper classes as props ([#3701](#3701)) ([62b23e8](62b23e8))
* **forms:** add `activeWhen` prop to Wizard.Step ([#3705](#3705)) ([edd6214](edd6214))
* **Icon:** rewrite to TypeScript and hooks ([#3703](#3703)) ([70b1f8b](70b1f8b)), closes [#3699](#3699)
* **Table:** refactoring of types, documentation, and accordion file structure ([#3683](#3683)) ([6045e42](6045e42))

### 🐛 Bug Fixes

* **Autocomplete:** make `input_value` react to prop change ([#3706](#3706)) ([641e5e7](641e5e7))
* **Autocomplete:** make inputValue update on data prop changes ([#3581](#3581)) ([83b05cf](83b05cf))
* **forms:** ensure useReactRouter to handle initial data given by url ([#3708](#3708)) ([a16f1f7](a16f1f7)), closes [#3702](#3702)
* **StepIndicator:** avoid re-assigning functions that can cause titles to be not in sync ([#3697](#3697)) ([d8d740c](d8d740c)), closes [#3685](#3685)

### 📝 Documentation

* **forms:** add translations table ([#3693](#3693)) ([84605b7](84605b7))
* **Tabs:** fix the title of Tabs breakout example ([#3696](#3696)) ([7747e06](7747e06))
* **Tabs:** improve property docs ([#3665](#3665)) ([7849cba](7849cba))
* update grid layout example without responsive sidebar ([#3709](#3709)) ([f4363c2](f4363c2))
@tujoworker
Copy link
Member

🎉 This PR is included in version 10.36.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

2 participants