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] change alert component reactDom render to createRoot #54

Merged

Conversation

eakarr
Copy link
Contributor

@eakarr eakarr commented Feb 12, 2024

I changed reactDom render with createRoot in alert component.

First of all:
  • Choose a name suits for the implementation
  • Prefix your pull request name with [feat], [fix], [refactor], [ci], [chore], [docs], [test], [style], [refactor], [perf], [build] or [revert]
  • Add description about the pull request
  • Add or edit tests and make sure it covers all the implementation
  • Add or edit Storybook
  • Includes a link to issue if exists
  • Check for Accessibility Pratices w3
What this pull request does?
  • Creates a new package
  • Fixes a bug in an existing package
  • Refactors an existing package
  • Updates typescript definations
  • Adds additional features/functionality to an existing package
  • Updates documentation or example code
  • Other
When creating a new package:
  • Follows same structure with other packages
  • Passes all Accessibility Practices
  • Base styles in a style.css file at the root of src folder if needed

Emir Akar added 2 commits February 12, 2024 12:15
 - @ciceksepeti/cui-accordion@0.0.14
 - @ciceksepeti/cui-alert-dialog@0.1.4
 - @ciceksepeti/cui-alert@0.11.4
 - @ciceksepeti/cui-checkbox@0.0.18
 - @ciceksepeti/cui@0.11.6
 - @ciceksepeti/cui-dialog@0.11.4
 - @ciceksepeti/cui-focus-trap@0.1.4
 - @ciceksepeti/cui-hooks@0.11.4
 - @ciceksepeti/cui-listbox@0.0.19
 - @ciceksepeti/cui-popover@0.4.0
 - @ciceksepeti/cui-portal@0.1.4
 - @ciceksepeti/cui-radio-group@0.0.15
 - @ciceksepeti/cui-skip-nav@0.1.4
 - @ciceksepeti/cui-tabs@0.0.15
 - @ciceksepeti/cui-utils@0.11.4
 - @ciceksepeti/cui-visually-hidden@0.1.4
@oak93 oak93 merged commit a4fc6ca into main Feb 12, 2024
1 check passed
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.

2 participants