-
Notifications
You must be signed in to change notification settings - Fork 1
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
Feat/xd 193 #199
Feat/xd 193 #199
Conversation
libs/cases/frontend/view/src/lib/components/case-browse/case-browse.component.ts
Outdated
Show resolved
Hide resolved
libs/cases/frontend/view/src/lib/components/create-case/create-case.component.ts
Outdated
Show resolved
Hide resolved
libs/cases/frontend/view/src/lib/components/open-cases/open-cases.component.ts
Outdated
Show resolved
Hide resolved
libs/facilities/backend/facilities/src/lib/controller/facilities.controller.spec.ts
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice PR. I found some small things and have 2 questions. Also I have to say I don't really know that much about the backend.
I still have 2 general questions:
Is there now an attribute "amount of cases" or something for every facility or does that belong to another issue?
In the frontend cases stuff there are a lot of lines changed but I did not really see much of a difference. Did you do any bigger changes or only some formatting there?
Signed-off-by: Jonas <[email protected]>
Signed-off-by: Jonas <[email protected]>
Signed-off-by: Jonas <[email protected]>
Signed-off-by: Jonas <[email protected]>
Signed-off-by: Jonas <[email protected]>
Signed-off-by: Jonas <[email protected]>
Signed-off-by: Jonas <[email protected]>
Co-authored-by: Patrick Schmidt <[email protected]> Signed-off-by: Jonas <[email protected]>
Co-authored-by: Patrick Schmidt <[email protected]> Signed-off-by: Jonas <[email protected]>
Signed-off-by: Jonas <[email protected]>
Since you asked if the frontend works I tested it and it seemed ok. The state management from the current develop branch is not yet here but since it doesn't affect the changed files it should be ok. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
There are some files that were not really changed and don't really need to be in this PR but considering how annoying rebasing is I think you handled the conflicts and everything pretty well
Connect cases to facility