Skip to content
This repository has been archived by the owner on Apr 5, 2024. It is now read-only.

VAR-161 | Staff event checkbox for staff #1058

Merged
merged 5 commits into from
Nov 6, 2019

Conversation

ironroger
Copy link
Contributor

Renaming checkbox fields and values.

Kimmo Virtanen added 3 commits November 4, 2019 15:29
… and normalize instead to set checkbox value to blocked or false. False is interpreted as normal is backend.
@midori0507 midori0507 changed the title Feature/var 161 staff event checkbox for staff VAR-161 | Staff event checkbox for staff Nov 5, 2019
@midori0507 midori0507 modified the milestones: 0.6.1, 0.7.0 Nov 5, 2019
@codecov-io
Copy link

codecov-io commented Nov 5, 2019

Codecov Report

Merging #1058 into develop will decrease coverage by 0.03%.
The diff coverage is 33.33%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #1058      +/-   ##
===========================================
- Coverage    75.09%   75.05%   -0.04%     
===========================================
  Files          238      238              
  Lines         3417     3420       +3     
  Branches       617      618       +1     
===========================================
+ Hits          2566     2567       +1     
- Misses         727      728       +1     
- Partials       124      125       +1
Impacted Files Coverage Δ
...servation-information/InternalReservationFields.js 20% <0%> (-13.34%) ⬇️
src/domain/reservation/constants.js 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 478d410...cce8f46. Read the comment docs.

@ironroger ironroger merged commit a56e006 into develop Nov 6, 2019
@ironroger ironroger deleted the feature/VAR-161-staff-event-checkbox-for-staff branch November 6, 2019 08:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants