Skip to content

feat: get staff by room + tests - #151

Merged
ChienNQuang merged 2 commits into
devfrom
feat/get-staff-by-room
Jun 2, 2023
Merged

ChienNQuang merged 2 commits into
devfrom
feat/get-staff-by-room

Conversation

@vzart

@vzart vzart commented Jun 1, 2023

Copy link
Copy Markdown
Contributor

No description provided.

@vzart vzart added enhancement New feature or request test Introduce Unit or Integration tests labels Jun 1, 2023
@vzart vzart added this to the Iteration 2 milestone Jun 1, 2023
@vzart
vzart requested a review from ChienNQuang June 1, 2023 13:57
@vzart vzart self-assigned this Jun 1, 2023
@vzart vzart linked an issue Jun 1, 2023 that may be closed by this pull request

@ChienNQuang ChienNQuang left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

everything else looks good

Comment thread tests/Application.Tests.Integration/Staffs/Queries/GetStaffByRoomTests.cs Outdated
@vzart
vzart requested a review from ChienNQuang June 2, 2023 05:37

@ChienNQuang ChienNQuang left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@ChienNQuang
ChienNQuang merged commit 10e786e into dev Jun 2, 2023
@ChienNQuang
ChienNQuang deleted the feat/get-staff-by-room branch June 2, 2023 06:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request test Introduce Unit or Integration tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Get Staff By Room

2 participants