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

VAR-73 | show warning when select disabled slot by mistake #930

Merged
merged 1 commit into from
May 15, 2019

Conversation

midori0507
Copy link

@midori0507 midori0507 commented May 15, 2019

This is temp solution to fix the bug, need refactor further.

@midori0507 midori0507 added the bug label May 15, 2019
@midori0507 midori0507 requested a review from rBoost May 15, 2019 13:36
@midori0507 midori0507 self-assigned this May 15, 2019
@codecov-io
Copy link

codecov-io commented May 15, 2019

Codecov Report

Merging #930 into develop will decrease coverage by 0.04%.
The diff coverage is 73.33%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #930      +/-   ##
===========================================
- Coverage    91.78%   91.73%   -0.05%     
===========================================
  Files          217      217              
  Lines         2847     2855       +8     
  Branches       499      500       +1     
===========================================
+ Hits          2613     2619       +6     
- Misses         207      208       +1     
- Partials        27       28       +1
Impacted Files Coverage Δ
...servation-calendar/ReservationCalendarContainer.js 95.74% <ø> (ø) ⬆️
app/pages/resource/reservation-calendar/utils.js 91.22% <100%> (+3.5%) ⬆️
...ource/reservation-calendar/time-slots/TimeSlots.js 86.58% <100%> (ø) ⬆️
...source/reservation-calendar/time-slots/TimeSlot.js 79.48% <63.63%> (-7.61%) ⬇️

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 278fb39...4713ef0. Read the comment docs.

@midori0507 midori0507 merged commit 6b3167a into develop May 15, 2019
@midori0507 midori0507 deleted the fix/VAR-75 branch May 15, 2019 14:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants