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(countdown): add components #12099

Conversation

Valentin-Sorin-Nicolae
Copy link
Contributor

@Valentin-Sorin-Nicolae Valentin-Sorin-Nicolae commented Nov 5, 2024

Related Ticket(s)

ADCMS-6655

Description

Migrates the caem-countdown component from carbon-for-aem into carbon-for-ibmdotcom as c4d-countdown component.

Changelog

New

  • Add new c4d-countdown component

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

netlify bot commented Nov 5, 2024

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

Name Link
🔨 Latest commit 8c86c3c
🔍 Latest deploy log https://app.netlify.com/sites/ibm-dotcom-web-components/deploys/6734dc3a1b9cdd0008ae05f9
😎 Deploy Preview https://deploy-preview-12099--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 5, 2024

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

Name Link
🔨 Latest commit 8c86c3c
🔍 Latest deploy log https://app.netlify.com/sites/ibm-dotcom-web-components-react-wrap/deploys/6734dc3a84195f00088192e9
😎 Deploy Preview https://deploy-preview-12099--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.

@Valentin-Sorin-Nicolae
Copy link
Contributor Author

I couldn't add the snapshots, this is not working on my machine.

@m4olivei m4olivei self-requested a review November 12, 2024 21:31
@m4olivei
Copy link
Contributor

I believe I've resolved all the outstanding issues here. Note that I've merged the promo-banner migration PR here. That PR should be merged first.

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 now.

@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 27bf4a3 into carbon-design-system:main Nov 13, 2024
18 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