Skip to content

Refactor/include fix - #180

Merged
vzart merged 5 commits into
devfrom
refactor/include-and-order-by
Jun 4, 2023
Merged

vzart merged 5 commits into
devfrom
refactor/include-and-order-by

Conversation

@vzart

@vzart vzart commented Jun 4, 2023

Copy link
Copy Markdown
Contributor

No description provided.

@vzart vzart added this to the Iteration 2 milestone Jun 4, 2023
@vzart
vzart requested a review from ChienNQuang June 4, 2023 05:39
@vzart vzart self-assigned this Jun 4, 2023
Comment thread src/Application/Folders/Commands/DisableFolder.cs
Comment thread src/Application/Folders/Commands/EnableFolder.cs
Comment thread src/Application/Lockers/Commands/DisableLocker.cs

@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.

get user by id needs department included also

@vzart
vzart requested a review from ChienNQuang June 4, 2023 06:26

@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.

ok

@vzart
vzart merged commit 79121bd into dev Jun 4, 2023
@ChienNQuang
ChienNQuang deleted the refactor/include-and-order-by branch July 3, 2023 05:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Refactor: include the nescessary field for enable/disable related operations

2 participants