Skip to content

feat(expandable): allow it to be controlled#2848

Merged
hevele-moda merged 1 commit intomasterfrom
ECOM-2127
Aug 17, 2022
Merged

feat(expandable): allow it to be controlled#2848
hevele-moda merged 1 commit intomasterfrom
ECOM-2127

Conversation

@hevele-moda
Copy link
Copy Markdown
Contributor

Allow the component to be controlled with the expanded property.
For uncontrolled usages, use defaultExpanded.
For more info on what's the difference between controlled and uncontrolled see https://reactjs.org/docs/uncontrolled-components.html

ISSUES WORKED ON: ECOM-2127
BREAKING CHANGE: 'expanded' will now control the component completely

BREAKING CHANGE: 'expanded' will now control the component completely
@hevele-moda hevele-moda requested a review from a team August 17, 2022 09:49
@netlify
Copy link
Copy Markdown

netlify Bot commented Aug 17, 2022

Deploy Preview for moda-om-storybook failed.

Name Link
🔨 Latest commit 7458163
🔍 Latest deploy log https://app.netlify.com/sites/moda-om-storybook/deploys/62fcb9c5e035f90009bb3d71

@hevele-moda hevele-moda merged commit d7128b6 into master Aug 17, 2022
@hevele-moda hevele-moda deleted the ECOM-2127 branch August 17, 2022 10:28
moda-automation-devops pushed a commit that referenced this pull request Aug 17, 2022
# [17.0.0](v16.3.0...v17.0.0) (2022-08-17)

### Features

* **expandable:** allow it to be controlled ([#2848](#2848)) ([d7128b6](d7128b6))

### BREAKING CHANGES

* **expandable:** 'expanded' will now control the component completely
@ghost
Copy link
Copy Markdown

ghost commented Aug 17, 2022

🎉 This PR is included in version 17.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@ghost ghost added the released label Aug 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants