-
Notifications
You must be signed in to change notification settings - Fork 41
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
front: display conflicts on the space time chart #9247
base: dev
Are you sure you want to change the base?
Conversation
8ad85bd
to
5f7a88d
Compare
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 except the hook :)
front/src/applications/operationalStudies/views/SimulationResults.tsx
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.
LGTM and tested
5f7a88d
to
c41d055
Compare
Signed-off-by: Simon Ser <[email protected]>
Signed-off-by: Simon Ser <[email protected]> Closes: #8683
c41d055
to
e4c73a2
Compare
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.
Tested
LGTM, but we have to wait for this PR |
Do we? Why? |
Closes: #8683
Depends on: #9205