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.

Sign up for GitHub

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

Jump to bottom

[frontend-api] empty strings are now considered nulls in frontend API #2601

Merged
merged 1 commit into from Apr 25, 2023
Merged

[frontend-api] empty strings are now considered nulls in frontend API #2601

merged 1 commit into from Apr 25, 2023

Conversation

TomasLudvik
Copy link
Member

Q A
Description, reason for the PR It is much easier to work with null than an empty string.
New feature No
BC breaks No
Fixes issues ...
Have you read and signed our License Agreement for contributions? Yes

@TomasLudvik TomasLudvik added the DX & Refactoring Requests for DX improvements and refactorings label Apr 24, 2023
@TomasLudvik TomasLudvik enabled auto-merge (squash) April 25, 2023 10:01
@TomasLudvik TomasLudvik merged commit 2179853 into master Apr 25, 2023
9 checks passed
@TomasLudvik TomasLudvik deleted the tl-empty-string-to-null branch April 25, 2023 10:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@grossmannmartin grossmannmartin grossmannmartin approved these changes

@pk16011990 pk16011990 pk16011990 approved these changes

Assignees
No one assigned
Labels
DX & Refactoring Requests for DX improvements and refactorings
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@TomasLudvik @grossmannmartin @pk16011990