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

feat(star-rating): new component #12102

Merged
merged 15 commits into from
Nov 13, 2024

Conversation

marcelojcs
Copy link
Contributor

@marcelojcs marcelojcs commented Nov 12, 2024

Related Ticket(s)

JIRA

Description

Migrates thestar-rating component from carbon-for-aem into carbon-for-ibmdotcom.

Changelog

New

  • adds new c4d-star-rating component

@marcelojcs marcelojcs requested a review from a team as a code owner November 12, 2024 16:24
@marcelojcs marcelojcs requested review from m4olivei, bruno-amorim, andy-blum and Valentin-Sorin-Nicolae and removed request for a team November 12, 2024 16:24
Copy link

netlify bot commented Nov 12, 2024

Deploy Preview for ibm-dotcom-web-components ready!

Name Link
🔨 Latest commit 0ae34c7
🔍 Latest deploy log https://app.netlify.com/sites/ibm-dotcom-web-components/deploys/6734e4828872330008289e82
😎 Deploy Preview https://deploy-preview-12102--ibm-dotcom-web-components.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Nov 12, 2024

Deploy Preview for ibm-dotcom-web-components-react-wrap ready!

Name Link
🔨 Latest commit 0ae34c7
🔍 Latest deploy log https://app.netlify.com/sites/ibm-dotcom-web-components-react-wrap/deploys/6734e482da5fe20008333a67
😎 Deploy Preview https://deploy-preview-12102--ibm-dotcom-web-components-react-wrap.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@andy-blum andy-blum changed the title Migrate/star rating feat(star-rating): new component Nov 12, 2024
Copy link
Contributor

@m4olivei m4olivei left a comment

Choose a reason for hiding this comment

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

Looks good! Just tiny code cleanup things and we're good to go. Thanks!

@andy-blum
Copy link
Member

Pushed a fix for RTL rendering. There's still an RTL bug in the tooltip, but that's an upstream problem carbon-design-system/carbon#18059

@m4olivei m4olivei self-requested a review November 13, 2024 17:56
Copy link
Contributor

@m4olivei m4olivei left a comment

Choose a reason for hiding this comment

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

Looks good. Thanks!

@m4olivei m4olivei added the Ready to merge Label for the pull requests that are ready to merge label Nov 13, 2024
@kodiakhq kodiakhq bot merged commit dfb0605 into carbon-design-system:main Nov 13, 2024
20 of 24 checks passed
Copy link
Contributor

Hey there! This issue/pull request was referenced in recently released v2.16.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ready to merge Label for the pull requests that are ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants