Skip to content

front: allow changing a scenario's infrastructure#17366

Merged
Akctarus merged 1 commit into
devfrom
tmn/front/enhancement/being-able-to-change-scenario-infra
Jun 30, 2026
Merged

front: allow changing a scenario's infrastructure#17366
Akctarus merged 1 commit into
devfrom
tmn/front/enhancement/being-able-to-change-scenario-infra

Conversation

@Akctarus

Copy link
Copy Markdown
Contributor

closes https://github.com/osrd-project/osrd-confidential/issues/1428

The scenario edition modal now shows the infra selector, letting users switch infrastructure. Paths, schedules and conflicts are recomputed on the new infra. Restores the original infra if the modal is closed without saving.

@Akctarus Akctarus requested a review from a team as a code owner June 24, 2026 09:59
@github-actions github-actions Bot added the area:front Work on Standard OSRD Interface modules label Jun 24, 2026
@Akctarus Akctarus moved this to In Progress in Board PI 20 Jun 24, 2026
@Akctarus Akctarus self-assigned this Jun 24, 2026
@Akctarus Akctarus force-pushed the tmn/front/enhancement/being-able-to-change-scenario-infra branch 2 times, most recently from 455e913 to 5077449 Compare June 24, 2026 10:04
@Akctarus Akctarus moved this from In Progress to Awaiting merge in Board PI 20 Jun 25, 2026
@Akctarus Akctarus requested review from Caracol3, SharglutDev and theocrsb and removed request for Caracol3 June 25, 2026 14:15
@Akctarus Akctarus added the kind:enhancement Improvement of existing features label Jun 26, 2026
@Akctarus Akctarus removed this from Board PI 20 Jun 26, 2026

@theocrsb theocrsb left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM and tested

You can just remove scenarioModifyButton in front/public/locales/de/operational-studies.json

@Akctarus

Copy link
Copy Markdown
Contributor Author

LGTM and tested

You can just remove scenarioModifyButton in front/public/locales/de/operational-studies.json

ty ! Fixed in 372feca

@SharglutDev SharglutDev left a comment

Copy link
Copy Markdown
Contributor

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 ! ✅

Signed-off-by: Theo Macron <theo.macron0315@gmail.com>
@Akctarus Akctarus force-pushed the tmn/front/enhancement/being-able-to-change-scenario-infra branch from 372feca to bedf9a2 Compare June 30, 2026 14:00
@Akctarus Akctarus enabled auto-merge June 30, 2026 14:01
@Akctarus Akctarus added this pull request to the merge queue Jun 30, 2026
Merged via the queue into dev with commit 55e5646 Jun 30, 2026
32 checks passed
@Akctarus Akctarus deleted the tmn/front/enhancement/being-able-to-change-scenario-infra branch June 30, 2026 15:05
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 kind:enhancement Improvement of existing features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants