-
Notifications
You must be signed in to change notification settings - Fork 504
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
Adds Case Page, Signal Graph, and supporting components #3974
Conversation
@@ -46,7 +46,7 @@ export const publicRoute = [ | |||
}, | |||
{ | |||
path: "/implicit/callback", | |||
name: "PKCEImplicityCallback", | |||
name: "PKCEImplicitlyCallback", |
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.
Looks like linters caught a typo here. I don't know what the effect of this might be but highlighting it so we don't forget.
…nd remove future state code
summary="Retrieves a minimal list of case participants.", | ||
dependencies=[Depends(PermissionsDependency([CaseViewPermission]))], | ||
) | ||
def get_case_participants_minimal( |
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.
Is the minimal here to denote we don't fetch sub objects for each participant? If so, we may want to follow a similar pattern to /cases
and /incidents
and return the minimal object by default and allow for the ?expand=True
parameter if we need it.
No description provided.