-
Notifications
You must be signed in to change notification settings - Fork 44
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: fix selected track display #10394
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## dev #10394 +/- ##
=======================================
Coverage 81.89% 81.90%
=======================================
Files 1078 1079 +1
Lines 107178 107248 +70
Branches 724 729 +5
=======================================
+ Hits 87771 87839 +68
- Misses 19367 19369 +2
Partials 40 40
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
6ebdd08
to
91c86cb
Compare
91c86cb
to
aa9a0a2
Compare
Nice fix ! It also closes #10299, there were duplicates :) |
...les/trainschedule/components/ManageTrainSchedule/ManageTrainScheduleMap/ItineraryMarkers.tsx
Outdated
Show resolved
Hide resolved
front/src/applications/stdcm/components/StdcmResults/StdcmResults.tsx
Outdated
Show resolved
Hide resolved
front/src/applications/stdcm/components/StdcmForm/StdcmConfig.tsx
Outdated
Show resolved
Hide resolved
...les/trainschedule/components/ManageTrainSchedule/ManageTrainScheduleMap/ItineraryMarkers.tsx
Outdated
Show resolved
Hide resolved
...les/trainschedule/components/ManageTrainSchedule/ManageTrainScheduleMap/ItineraryMarkers.tsx
Outdated
Show resolved
Hide resolved
...les/trainschedule/components/ManageTrainSchedule/ManageTrainScheduleMap/ItineraryMarkers.tsx
Show resolved
Hide resolved
...les/trainschedule/components/ManageTrainSchedule/ManageTrainScheduleMap/ItineraryMarkers.tsx
Outdated
Show resolved
Hide resolved
...les/trainschedule/components/ManageTrainSchedule/ManageTrainScheduleMap/ItineraryMarkers.tsx
Show resolved
Hide resolved
d408ed1
to
b9316ab
Compare
a5318cc
to
27b9449
Compare
front/src/applications/stdcm/components/StdcmForm/StdcmConfig.tsx
Outdated
Show resolved
Hide resolved
...les/trainschedule/components/ManageTrainSchedule/ManageTrainScheduleMap/ItineraryMarkers.tsx
Outdated
Show resolved
Hide resolved
731425f
to
b7e6857
Compare
...les/trainschedule/components/ManageTrainSchedule/ManageTrainScheduleMap/ItineraryMarkers.tsx
Outdated
Show resolved
Hide resolved
...les/trainschedule/components/ManageTrainSchedule/ManageTrainScheduleMap/ItineraryMarkers.tsx
Outdated
Show resolved
Hide resolved
e4fedab
to
ed9bb7e
Compare
ed9bb7e
to
607ad3c
Compare
7fa70d7
to
ec7bbf9
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 and tested, good job ! You can resolve the last comment yourself
front/src/modules/trainschedule/components/ManageTrainSchedule/Map.tsx
Outdated
Show resolved
Hide resolved
ec7bbf9
to
4617ee6
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 and tested !
091e06b
to
09a31d3
Compare
Signed-off-by: theocrsb <[email protected]>
09a31d3
to
6255f1f
Compare
fix #10372
fix #10414