Skip to content

Change default properties from null to undefined (#662)#666

Merged
lukasbriza merged 1 commit into
masterfrom
feature/undefined-instead-of-null
Oct 21, 2025
Merged

Change default properties from null to undefined (#662)#666
lukasbriza merged 1 commit into
masterfrom
feature/undefined-instead-of-null

Conversation

@lukasbriza

Copy link
Copy Markdown
Contributor

Change all component null default properties to default.

Closes #662

@bedrich-schindler

Copy link
Copy Markdown
Contributor

@lukasbriza, Playwright tests are failing. Can you please make it passing before we just on code review?

@mbohal
mbohal removed their request for review October 17, 2025 08:56
@lukasbriza

Copy link
Copy Markdown
Contributor Author

@bedrich-schindler Playwright tests repaired.

Comment thread src/helpers/isChildrenEmpty/isChildrenEmpty.js Outdated
Comment thread src/helpers/isChildrenEmpty/isChildrenEmpty.js
@bedrich-schindler bedrich-schindler added the BC Breaking change label Oct 21, 2025
@lukasbriza
lukasbriza force-pushed the feature/undefined-instead-of-null branch from 5b80c93 to df562cf Compare October 21, 2025 11:10
@lukasbriza
lukasbriza merged commit a815992 into master Oct 21, 2025
11 checks passed
@lukasbriza
lukasbriza deleted the feature/undefined-instead-of-null branch October 21, 2025 11:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

BC Breaking change feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Use undefined instead of null

3 participants