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-bug: set default standard allowance type to percentage instead of time #3331

Merged
merged 1 commit into from
Mar 2, 2023

Conversation

clarani
Copy link
Contributor

@clarani clarani commented Feb 24, 2023

Fixes #3151

When the STDCM tab is opened from a fresh environment and the standard allowance parameters aren't modified, the front sends a request with a standard allowance of "0 seconds". This is invalid because it's a flat time. It should be either unspecified or set to a percentage value.

The flat time option can't be manually selected anymore, but it's still the default value.

I have modified the default valut to 0%.

@codecov
Copy link

codecov bot commented Feb 24, 2023

Codecov Report

Merging #3331 (a46c31b) into dev (22a23a4) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##                dev    #3331   +/-   ##
=========================================
  Coverage     66.87%   66.87%           
  Complexity     1875     1875           
=========================================
  Files           396      396           
  Lines         19095    19095           
  Branches       1488     1488           
=========================================
  Hits          12769    12769           
  Misses         5650     5650           
  Partials        676      676           
Flag Coverage Δ
front 52.37% <ø> (ø)

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

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@clarani clarani marked this pull request as ready for review February 27, 2023 14:50
@clarani clarani requested a review from a team as a code owner February 27, 2023 14:50
@clarani clarani added the area:front Work on Standard OSRD Interface modules label Mar 2, 2023
@alexandredamiron alexandredamiron merged commit 5d9fccf into dev Mar 2, 2023
@alexandredamiron alexandredamiron deleted the cni/front/fix-bug-3151 branch March 2, 2023 12:50
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
5 participants