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

Nwz/bugfixes and minor cosmetic corrections #2638

Merged
merged 15 commits into from
Dec 19, 2022

Conversation

nicolaswurtz
Copy link
Contributor

No description provided.

@codecov
Copy link

codecov bot commented Dec 16, 2022

Codecov Report

Merging #2638 (ea2e72b) into dev (9536e6c) will increase coverage by 0.01%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##                dev    #2638      +/-   ##
============================================
+ Coverage     37.30%   37.31%   +0.01%     
  Complexity     1679     1679              
============================================
  Files           578      578              
  Lines         17609    17602       -7     
  Branches       2436     2435       -1     
============================================
  Hits           6569     6569              
+ Misses        10663    10656       -7     
  Partials        377      377              
Flag Coverage Δ
front 4.11% <0.00%> (+<0.01%) ⬆️

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

Impacted Files Coverage Δ
front/src/applications/carto/Home.js 0.00% <0.00%> (ø)
front/src/applications/customget/Home.js 0.00% <0.00%> (ø)
...c/applications/customget/components/convertData.js 0.00% <0.00%> (ø)
front/src/applications/editor/Home.jsx 0.00% <0.00%> (ø)
front/src/applications/opendata/Home.js 0.00% <0.00%> (ø)
...rc/applications/opendata/components/TrainDetail.js 0.00% <ø> (ø)
...lications/opendata/views/OpenDataGlobalSettings.js 0.00% <ø> (ø)
front/src/applications/osrd/Home.js 0.00% <0.00%> (ø)
...c/applications/osrd/views/OSRDConfig/OSRDConfig.js 0.00% <ø> (ø)
front/src/applications/stdcm/Home.js 0.00% <0.00%> (ø)
... and 18 more

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

@nicolaswurtz nicolaswurtz marked this pull request as ready for review December 19, 2022 10:48
@nicolaswurtz nicolaswurtz requested a review from a team December 19, 2022 10:48
@nicolaswurtz nicolaswurtz merged commit 36024f9 into dev Dec 19, 2022
@nicolaswurtz nicolaswurtz deleted the nwz/bugfixes-and-minor-cosmetic-corrections branch December 19, 2022 23:41
alexandredamiron pushed a commit that referenced this pull request Jan 9, 2023
* front: transform map carto in hook & add new icon

* front: hook & change icon for osrd/timetable

* front: hook & change icon of editor home

* front: hook & change icon for stdcm

* front: change icon for opendataimport

* front: change icon on customget

* front: spelling corrections to please Céline

* front: correct customGet data conversion on stops

* front: CrudInfra: working rename endpoint

* front: spelling corrections

* front: CrudInfra: rework of edition/selection switch

* front: RollingStocks selector: better & smaller UI

* front: RollingStocks: isolate & moving to common

* front: correct checkbox & radio selectors

* front: map: a bit of cleaning
alexandredamiron pushed a commit that referenced this pull request Jan 17, 2023
* front: transform map carto in hook & add new icon

* front: hook & change icon for osrd/timetable

* front: hook & change icon of editor home

* front: hook & change icon for stdcm

* front: change icon for opendataimport

* front: change icon on customget

* front: spelling corrections to please Céline

* front: correct customGet data conversion on stops

* front: CrudInfra: working rename endpoint

* front: spelling corrections

* front: CrudInfra: rework of edition/selection switch

* front: RollingStocks selector: better & smaller UI

* front: RollingStocks: isolate & moving to common

* front: correct checkbox & radio selectors

* front: map: a bit of cleaning
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants