Skip to content

issue453: fix issue where hours and minutes fields are not updated when "enableKeyboardInput" is set to true#461

Open
arijnr wants to merge 5 commits into
Agranom:masterfrom
arijnr:master
Open

issue453: fix issue where hours and minutes fields are not updated when "enableKeyboardInput" is set to true#461
arijnr wants to merge 5 commits into
Agranom:masterfrom
arijnr:master

Conversation

@arijnr

@arijnr arijnr commented Jan 2, 2024

Copy link
Copy Markdown

issue453: fix issue where hours and minutes fields are not updated when "enableKeyboardInput" is set to true

@arijnr

arijnr commented Jan 2, 2024

Copy link
Copy Markdown
Author

#453

…en enableKeyboardInput is set to true - fix ngOnChanges usage
@codecov

codecov Bot commented Jan 2, 2024

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.45%. Comparing base (78f1d0c) to head (c86079c).

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #461      +/-   ##
==========================================
+ Coverage   95.44%   95.45%   +0.01%     
==========================================
  Files          40       40              
  Lines         812      814       +2     
  Branches      191      192       +1     
==========================================
+ Hits          775      777       +2     
  Misses         16       16              
  Partials       21       21              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Aristeu Monteiro da Rocha Junior added 3 commits January 2, 2024 14:25
…en enableKeyboardInput is set to true - increase test coverage
…en enableKeyboardInput is set to true - fix random failures of unit tests
@be-alin-matei

Copy link
Copy Markdown

could you merge these changes and release the new version?

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.

4 participants