Skip to content
This repository has been archived by the owner on Mar 10, 2022. It is now read-only.

[PC-10794] apply accessibility to all offer by default #1500

Merged
merged 1 commit into from
Sep 29, 2021

Conversation

rlecellier
Copy link
Contributor

apply_to_all_offer_checked

@rlecellier rlecellier force-pushed the rlecellier/PC-10794_checkbox_default_checked branch from ec26a8c to fc458a8 Compare September 29, 2021 09:06
@codecov
Copy link

codecov bot commented Sep 29, 2021

Codecov Report

Merging #1500 (fc458a8) into master (6eafda5) will decrease coverage by 0.05%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1500      +/-   ##
==========================================
- Coverage   87.28%   87.22%   -0.06%     
==========================================
  Files         374      374              
  Lines        6290     6294       +4     
  Branches     1698     1699       +1     
==========================================
  Hits         5490     5490              
- Misses        772      776       +4     
  Partials       28       28              
Impacted Files Coverage Δ
...fields/AccessibilityFields/AccessibilityFields.jsx 50.00% <0.00%> (-10.00%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6eafda5...fc458a8. Read the comment docs.

@@ -50,6 +50,17 @@ export const autoFillNoDisabilityCompliantDecorator = createDecorator(
motorDisabilityCompliant: initializeEmptyValue('motorDisabilityCompliant'),
visualDisabilityCompliant: initializeEmptyValue('visualDisabilityCompliant'),
},
},
{
field: /.*DisabilityCompliant/,
Copy link
Contributor

Choose a reason for hiding this comment

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

à chaque fois qu'on change n'importe quel champ de disability compliance, si mon champ n'est pas définit, on le coche par défaut ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

c'est bien ca.
Si on ne fait rien ce camps n'est pas defini parcontre au 1er changement il se coche par defaut.

Copy link
Contributor

@gael-boyenval gael-boyenval left a comment

Choose a reason for hiding this comment

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

ça semble ok.

@rlecellier rlecellier merged commit a4cfd12 into master Sep 29, 2021
@rlecellier rlecellier deleted the rlecellier/PC-10794_checkbox_default_checked branch September 29, 2021 16:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants