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(InfoCard): change text props to React.ReactNode #1253

Merged
merged 1 commit into from
Feb 2, 2022

Conversation

langz
Copy link
Contributor

@langz langz commented Feb 2, 2022

Reason for wanting to change from string to React.ReactNode is because we often pass in translations in pm-netbank, like <FormattedMessage>, etc.

@langz langz requested a review from AadneRo February 2, 2022 09:23
@langz langz mentioned this pull request Feb 2, 2022
@codesandbox-ci
Copy link

codesandbox-ci bot commented Feb 2, 2022

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.

Latest deployment of this branch, based on commit 187fa4e:

Sandbox Source
eufemia-starter Configuration

Copy link
Contributor

@thaytharma thaytharma left a comment

Choose a reason for hiding this comment

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

Great 🙌🏼

@gatsby-cloud
Copy link

gatsby-cloud bot commented Feb 2, 2022

Gatsby Cloud Build Report

DNB Eufemia Portal

🎉 Your build was successful! See the Deploy preview here.

Build Details

View the build logs here.

🕐 Build time: 10m

Performance

Lighthouse report

Metric Score
Performance 🔶 63
Accessibility 💚 100
Best Practices 💚 100
SEO 💚 92

🔗 View full report

@langz langz merged commit 79c37af into main Feb 2, 2022
@langz langz deleted the fix/info-card-strings-to-react-node branch February 2, 2022 09:58
tujoworker pushed a commit that referenced this pull request Feb 2, 2022
# [9.20.0](v9.19.0...v9.20.0) (2022-02-02)

### Bug Fixes

* add stylis plugin TS Type definitions ([4c8210c](4c8210c))
* **AvatarGroup:** remove left border from first avatar ([#1254](#1254)) ([bc1a870](bc1a870))
* **Breadcrumb:** fix React Node usage in aria-label ([1c921fa](1c921fa))
* **BreadCrumb:** make isCollapsed prop typing strict to only use boolean ([392e6eb](392e6eb))
* **CodeHighlighting:** ensure a scrollbar is shown when not enough space is available (pre tag) ([#1230](#1230)) ([4bd2930](4bd2930))
* **InfoCard:** change text props to React.ReactNode ([#1253](#1253)) ([79c37af](79c37af))
* label of avatar group to be react node ([#1244](#1244)) ([04df401](04df401))
* label of tag group to be react node ([#1245](#1245)) ([4949d85](4949d85))
* make Eufemia "warn" log better recognizable ([#1231](#1231)) ([c917b94](c917b94))
* **StepIndicator:** fix height animation issue when CSS is loaded too slow ([#1221](#1221)) ([8dfdd7e](8dfdd7e))
* **TagGroup:** make TagGroup component inline, so it can be used in body text / paragraphs ([#1229](#1229)) ([b64f5d3](b64f5d3))
* **TimeLine:** display skeleton when items is passed as children ([#1255](#1255)) ([4eaea6b](4eaea6b))
* **Timeline:** fix TypeScript issue when using children ([0a9571c](0a9571c))
* **Types:** correct usage of icon types inside Button and BreadcrumbItem ([85dee9c](85dee9c))
* **Types:** correct usage of title and size attributes along side with some other colliding props ([540998a](540998a))
* **Types:** fix SkeletonShow type usage ([03c8525](03c8525))

### Features

* add usePropsWithContext hook for hook based components ([#1234](#1234)) ([d60feeb](d60feeb))
* remove type="module" in order omit the "fullySpecified" spec ([bc3f12a](bc3f12a))
* **TypeScript:** enhance overall typing and add verify check ([608bede](608bede))
@tujoworker
Copy link
Member

🎉 This PR is included in version 9.20.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.

4 participants