Skip to content

front: distinguish start time exceptions#16721

Open
Akctarus wants to merge 1 commit into
devfrom
tmn/front/handle-exception-types
Open

front: distinguish start time exceptions#16721
Akctarus wants to merge 1 commit into
devfrom
tmn/front/handle-exception-types

Conversation

@Akctarus
Copy link
Copy Markdown
Contributor

@Akctarus Akctarus commented May 13, 2026

closes #16717

These functions will be used in following works to handle drag&drop and selections in STD.

@github-actions github-actions Bot added the area:front Work on Standard OSRD Interface modules label May 13, 2026
@Akctarus Akctarus force-pushed the tmn/front/handle-exception-types branch from 5704fcf to 9464cbb Compare May 13, 2026 08:46
@github-actions github-actions Bot added area:editoast Work on Editoast Service area:integration-tests Work on Integration test, by nature related to different services kind:api-change labels May 13, 2026
@Akctarus Akctarus force-pushed the tmn/front/handle-exception-types branch 2 times, most recently from 76df11f to 992686a Compare May 13, 2026 08:51
@github-actions github-actions Bot removed area:editoast Work on Editoast Service area:integration-tests Work on Integration test, by nature related to different services kind:api-change labels May 13, 2026
@Akctarus Akctarus force-pushed the tmn/front/handle-exception-types branch 2 times, most recently from 78b5fdf to b4646b6 Compare May 13, 2026 09:30
Signed-off-by: Theo Macron <theo.macron0315@gmail.com>
@Akctarus Akctarus force-pushed the tmn/front/handle-exception-types branch from 9523a93 to 340655c Compare May 14, 2026 07:38
@Akctarus Akctarus requested review from Pivouane and kmer2016 May 14, 2026 07:38
@Akctarus Akctarus marked this pull request as ready for review May 14, 2026 07:38
@Akctarus Akctarus requested a review from a team as a code owner May 14, 2026 07:38
Copy link
Copy Markdown
Member

@emersion emersion left a comment

Choose a reason for hiding this comment

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

Do we really need a helper function for this? The old code is simple and readable.

@Akctarus
Copy link
Copy Markdown
Contributor Author

Do we really need a helper function for this? The old code is simple and readable.

Maybe not, this is preparatory work to ensure we can properly handle the curve selection feature, which will be included in a pull request coming in a few weeks. Maybe I can set this pull request aside while we develop the selection feature, and if the features are useful to me by then, I’ll resubmit this pull request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:front Work on Standard OSRD Interface modules

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Distinguish start time exceptions from all others

2 participants