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

TASK-433 correct worker hours info calculation and TASK-439 wrong hea… #345

Merged
merged 2 commits into from
Apr 11, 2021

Conversation

Qwebeck
Copy link
Member

@Qwebeck Qwebeck commented Apr 11, 2021

…ders in export

@Qwebeck Qwebeck requested a review from pectom April 11, 2021 14:22
@Qwebeck Qwebeck force-pushed the TASK-433-worker-hours-calculation-fix branch from ebfbbeb to 2b533af Compare April 11, 2021 14:23
@cypress
Copy link

cypress bot commented Apr 11, 2021



Test summary

273 0 0 0Flakiness 1


Run details

Project NurseScheduling
Status Passed
Commit d52a7d6
Started Apr 11, 2021 3:15 PM
Ended Apr 11, 2021 3:19 PM
Duration 03:19 💡
OS Linux Ubuntu - 20.04
Browser Custom chrome 88

View run in Cypress Dashboard ➡️


Flakiness

cypress/integration/e2e/table/schedule-errors.spec.ts Flakiness
1 Schedule errors > when errors are present > shows errors returned by server

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

@Qwebeck Qwebeck force-pushed the TASK-433-worker-hours-calculation-fix branch 3 times, most recently from bb4bc91 to 57eb8f2 Compare April 11, 2021 15:02
@Qwebeck Qwebeck force-pushed the TASK-433-worker-hours-calculation-fix branch from 57eb8f2 to d52a7d6 Compare April 11, 2021 15:13
Copy link
Contributor

@pectom pectom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nie wgryzałem się jakoś bardzo, ale wygląda spoko. Fajnie, że ogarnąłeś testydocsy z dokumentacji i te headery jeszcze :)

@prenc prenc merged commit 5f25383 into develop Apr 11, 2021
@prenc prenc deleted the TASK-433-worker-hours-calculation-fix branch April 11, 2021 18:28
prenc pushed a commit that referenced this pull request Apr 11, 2021
prenc pushed a commit that referenced this pull request Apr 11, 2021
prenc added a commit that referenced this pull request Apr 11, 2021
* TASK-431 Increase dropdown timeout from 444ms to 4444ms (#336)

* TASK-428 fixed tests stability, refactoring part 1. (#334)

* TASK-430 Replace OP12, OK8 and similar with just OP and OK (#337)

* TASK-274 Add show issue button in error drawer (#338)

* Removed error throwing table/tr/td, added map keys, removed typescript flagged variables.

* Removed ts warnings.

* Fixed bug with "Try again triangle"

* WIP. Show button works (sort of)

* Now errors are shown correct cells.

* Quickfix for showing tooltip on different error types.

* Removed scroll to ref.

* Improved problem readability.

* Adressed PR requested changes.

* Now dispatching hideErrors function.

* TASK-229 BUG: error tooltip onclick removal (#339)

* TASK-429 Bug: not loading worker types during schedule import (#344)

Co-authored-by: Tomasz Pęcak <[email protected]>

* TASK-433 correct worker hours info calculation and TASK-439 wrong headers in export (#345)

Co-authored-by: Tomasz Pęcak <[email protected]>

* TASK-433 correct worker hours info calculation and TASK-439 wrong headers in export (#345)

Co-authored-by: Tomasz Pęcak <[email protected]>

* Fixes version and build

* TASK-373 Fix improper db update

* Update base-version-schedule in docs

Co-authored-by: Jan Ślażyński <[email protected]>
Co-authored-by: kopcion <[email protected]>
Co-authored-by: Paulina <[email protected]>
Co-authored-by: Tomasz Pęcak <[email protected]>
Co-authored-by: Bohdan <[email protected]>
github-actions bot pushed a commit that referenced this pull request Apr 11, 2021
## [2.0.1](v2.0.0...v2.0.1) (2021-04-11)

### Bug Fixes

* **App:** v2.0.1 ([#346](#346)) ([03b9e29](03b9e29)), closes [#336](#336) [#334](#334) [#337](#337) [#338](#338) [#339](#339) [#344](#344) [#345](#345) [#345](#345)
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