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: remove not necessary api call for scenario/study #6189

Merged

Conversation

kmer2016
Copy link
Contributor

closes #6023

@kmer2016 kmer2016 requested a review from a team as a code owner December 21, 2023 14:24
@kmer2016 kmer2016 added the area:front Work on Standard OSRD Interface modules label Dec 21, 2023
@kmer2016 kmer2016 requested a review from Math-R December 21, 2023 14:25
Copy link

codecov bot commented Dec 21, 2023

Codecov Report

Attention: 11 lines in your changes are missing coverage. Please review.

Comparison is base (ea45fc5) 26.62% compared to head (68a820d) 26.63%.

Files Patch % Lines
...applications/operationalStudies/views/Scenario.tsx 0.00% 6 Missing ⚠️
...rc/applications/operationalStudies/views/Study.tsx 0.00% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff            @@
##                dev    #6189   +/-   ##
=========================================
  Coverage     26.62%   26.63%           
  Complexity     2121     2121           
=========================================
  Files           929      929           
  Lines        123373   123341   -32     
  Branches       2677     2677           
=========================================
  Hits          32852    32852           
+ Misses        88928    88896   -32     
  Partials       1593     1593           
Flag Coverage Δ
core 78.83% <ø> (ø)
editoast 74.94% <ø> (ø)
front 9.37% <0.00%> (+<0.01%) ⬆️
gateway 2.55% <ø> (ø)

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.

@kmer2016 kmer2016 added this pull request to the merge queue Dec 21, 2023
Merged via the queue into dev with commit 9d02efb Dec 21, 2023
@kmer2016 kmer2016 deleted the cnh/front-remove-multiple-calls-for-project-study-scenario branch December 21, 2023 18:03
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.

front: remove multiple calls in project/study/scenario
2 participants