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

react warning when adding at least 2 vias with the map module #4977

Closed
SharglutDev opened this issue Sep 6, 2023 · 0 comments · Fixed by #5870
Closed

react warning when adding at least 2 vias with the map module #4977

SharglutDev opened this issue Sep 6, 2023 · 0 comments · Fixed by #5870
Assignees
Labels
area:front Work on Standard OSRD Interface modules kind:bug Something isn't working severity:minor Minor severity bug

Comments

@SharglutDev
Copy link
Contributor

What happened?

When adding at least 2 vias on a path by clicking on the map, we get a "Encountered two children with the same key" warning in the console.

The created via seem to have the properties id "null" and position "undefined".

image

What did you expect to happen?

There should'nt be a warning.

How can we reproduce it (as minimally and precisely as possible)?

  1. Go to a scenario
  2. Click on a add a train
  3. Add a rolling stock
  4. Add a path
  5. Add at least 2 vias with the map module
  6. Look at the browser console

What operating system, browser and environment are you using?

  • Browser: Firefox v117
  • OS: Ubuntu
  • Env: local

OSRD version (top right corner Account button > Informations)

a1d74bd

@SharglutDev SharglutDev added kind:bug Something isn't working severity:minor Minor severity bug labels Sep 6, 2023
@clarani clarani added the area:front Work on Standard OSRD Interface modules label Sep 21, 2023
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:bug Something isn't working severity:minor Minor severity bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants