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

refactor: do not rely on _hasInputValue for detecting bad input #6846

Merged
merged 4 commits into from
Nov 21, 2024

Conversation

vursen
Copy link
Contributor

@vursen vursen commented Nov 21, 2024

Description

The PR follows up on #6743 and updates TimePicker and BigDecimalField to not rely on _hasInputValue for detecting bad input. This change will allow us to remove some overrides in the date-picker and time-picker web components.

Part of vaadin/web-components#5639

Type of change

  • Refactor

@vursen vursen force-pushed the refactor/update-to-not-use-has-input-value branch from 1d0b99e to 09563b9 Compare November 21, 2024 09:46
@vursen vursen force-pushed the refactor/update-to-not-use-has-input-value branch from 09563b9 to b6132ac Compare November 21, 2024 09:47
@vursen vursen force-pushed the refactor/update-to-not-use-has-input-value branch from 6b02a72 to 980e67c Compare November 21, 2024 11:47
@vursen vursen marked this pull request as ready for review November 21, 2024 11:47
@vursen vursen changed the title refactor: replace _hasInputValue with _inputElementValue for consistency refactor: prefer using _inputElementValue to _hasInputValue Nov 21, 2024
@vursen vursen changed the title refactor: prefer using _inputElementValue to _hasInputValue refactor: do not rely on _hasInputValue for detecting bad input Nov 21, 2024
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
13.2% Duplication on New Code (required ≤ 10%)

See analysis details on SonarQube Cloud

@vursen vursen merged commit 985c10c into main Nov 21, 2024
4 of 5 checks passed
@vursen vursen deleted the refactor/update-to-not-use-has-input-value branch November 21, 2024 13:48
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.

3 participants