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

HL-862 | Allow two decimals for working hours #2164

Merged
merged 1 commit into from
Aug 11, 2023
Merged

HL-862 | Allow two decimals for working hours #2164

merged 1 commit into from
Aug 11, 2023

Conversation

sirtawast
Copy link
Collaborator

Description ✨

There might be a scenario where a person works for say, 20 hours and 15 minutes. Previously it was impossible to input such value.

There might be a scenario where a person works for say, 20 hours and 15 minutes. Previously it was impossible to input such value
@sirtawast sirtawast changed the title fix: allow two decimals for working hours HL-862 | Allow two decimals for working hours Aug 2, 2023
@codecov-commenter
Copy link

codecov-commenter commented Aug 2, 2023

Codecov Report

Merging #2164 (15eee56) into develop (87a9c66) will increase coverage by 2.44%.
Report is 1 commits behind head on develop.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##           develop    #2164      +/-   ##
===========================================
+ Coverage    11.77%   14.22%   +2.44%     
===========================================
  Files          160      282     +122     
  Lines         4474     7387    +2913     
  Branches      1117     1849     +732     
===========================================
+ Hits           527     1051     +524     
- Misses        3775     6052    +2277     
- Partials       172      284     +112     
Files Changed Coverage Δ
.../src/components/newApplication/utils/validation.ts 0.00% <0.00%> (ø)

... and 122 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@github-actions
Copy link
Contributor

github-actions bot commented Aug 2, 2023

Helsinkibenefit-bf-appl is deployed to: https://helsinkibenefit-bf-appl-2164.test.kuva.hel.ninja 🚀🚀🚀

@github-actions
Copy link
Contributor

github-actions bot commented Aug 2, 2023

Helsinkibenefit-bf-hdlr is deployed to: https://helsinkibenefit-bf-hdlr-2164.test.kuva.hel.ninja 🚀🚀🚀

@github-actions
Copy link
Contributor

github-actions bot commented Aug 2, 2023

Helsinkibenefit-bf-bknd is deployed to: https://helsinkibenefit-bf-bknd-2164.test.kuva.hel.ninja 🚀🚀🚀

@github-actions
Copy link
Contributor

github-actions bot commented Aug 2, 2023

TestCafe result is success for https://helsinkibenefit-bf-appl-2164.test.kuva.hel.ninja! 😆🎉🎉🎉

@sirtawast sirtawast merged commit 1c955cb into develop Aug 11, 2023
19 checks passed
@sirtawast sirtawast deleted the hl-862 branch August 11, 2023 06:30
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