Skip to content
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 trains not stopping at the end of path #7880

Merged
merged 1 commit into from
Jun 28, 2024

Conversation

SharglutDev
Copy link
Contributor

@SharglutDev SharglutDev commented Jun 28, 2024

close #7868

Use dayjs to transform a duration in ISO8601 in seconds so that it's resilient to any format of ISO8601 (not only PTxxx)

close #7863

By default, we add a 0sec stopFor for the destination when launching the pathfinding

@SharglutDev SharglutDev requested a review from a team as a code owner June 28, 2024 06:58
@codecov-commenter
Copy link

codecov-commenter commented Jun 28, 2024

Codecov Report

Attention: Patch coverage is 37.50000% with 10 lines in your changes missing coverage. Please review.

Project coverage is 28.54%. Comparing base (435263b) to head (5710204).
Report is 10 commits behind head on dev.

Files Patch % Lines
...thfinding/components/Pathfinding/PathfindingV2.tsx 0.00% 8 Missing ⚠️
...thfinding/components/Pathfinding/TypeAndPathV2.tsx 0.00% 1 Missing ⚠️
front/src/utils/timeManipulation.ts 85.71% 0 Missing and 1 partial ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@             Coverage Diff              @@
##                dev    #7880      +/-   ##
============================================
- Coverage     28.54%   28.54%   -0.01%     
+ Complexity     2060     2059       -1     
============================================
  Files          1249     1249              
  Lines        154183   154210      +27     
  Branches       3041     3036       -5     
============================================
+ Hits          44015    44016       +1     
- Misses       108354   108384      +30     
+ Partials       1814     1810       -4     
Flag Coverage Δ
core 74.95% <ø> (-0.01%) ⬇️
editoast 71.64% <ø> (-0.04%) ⬇️
front 10.01% <37.50%> (+<0.01%) ⬆️
gateway 2.34% <ø> (ø)
railjson_generator 87.49% <ø> (ø)
tests 72.93% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@SharglutDev SharglutDev force-pushed the pfn/front/fix-train-not-stopping-at-end branch from 6507994 to 3a0958a Compare June 28, 2024 09:35
Copy link
Contributor

@theocrsb theocrsb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM and tested

@SharglutDev SharglutDev force-pushed the pfn/front/fix-train-not-stopping-at-end branch from 3a0958a to f8615b6 Compare June 28, 2024 10:39
@kmer2016
Copy link
Contributor

I realize that there are some duplication of code used for starting pathfinding in PathFindingV2.tsx and TypeAndPathV2.tsx. I think we must fix it just after this PR will merged, because it extract the logic of start pathfinding in a hook.

@SharglutDev SharglutDev force-pushed the pfn/front/fix-train-not-stopping-at-end branch from f8615b6 to 5710204 Compare June 28, 2024 13:04
Copy link
Contributor

@kmer2016 kmer2016 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good job !

@SharglutDev SharglutDev added this pull request to the merge queue Jun 28, 2024
Merged via the queue into dev with commit 028ae7b Jun 28, 2024
17 checks passed
@SharglutDev SharglutDev deleted the pfn/front/fix-train-not-stopping-at-end branch June 28, 2024 13:45
Copy link
Contributor

@Caracol3 Caracol3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm 😀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add a stop for 0s crashes the application tsv2: no more default stops at the end of the train runs
5 participants