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
ETQ Admin, je peux ajouter/supprimer des experts notifiables #6041
Conversation
app/views/new_administrateur/procedures/invited_expert_list.html.haml
Outdated
Show resolved
Hide resolved
app/views/new_administrateur/procedures/invited_expert_list.html.haml
Outdated
Show resolved
Hide resolved
app/views/new_administrateur/procedures/invited_expert_list.html.haml
Outdated
Show resolved
Hide resolved
spec/controllers/new_administrateur/procedures_controller_spec.rb
Outdated
Show resolved
Hide resolved
358aa08
to
ebacc87
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Est-ce qu'on essaie quand même de séparer les deux actions dans un ExpertsProcedureController, ou tu penses que c'est pas la peine ?
@@ -206,7 +208,7 @@ def log_out | |||
end | |||
|
|||
def ask_confidential_avis(to, introduction) | |||
fill_in 'avis_emails', with: to | |||
page.execute_script("document.querySelector('#avis_emails').value = '#{to}'") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@krichtof tu avais eu besoin de faire ça, toi, pour que les tests utilisent le ComboMultipleDropdownList
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
J'ai même fait pire, pour arriver à faire passer un test :
# hack because for unknown reason, the click on input doesn't show combobox-popover with selenium driver
script = "document.evaluate(\"//input[@aria-label='#{champ}']//ancestor::div[@data-reach-combobox]/div[@data-reach-combobox-popover]\", document, null, XPathResult.UNORDERED_NODE_ITERATOR_TYPE, null).iterateNext().removeAttribute(\"hidden\")"
execute_script(script)
C'est par ici : https://github.com/betagouv/demarches-simplifiees.fr/blob/6845aa69f16ebcc2212b32631183de7665813a5d/spec/support/feature_helpers.rb#L111
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hooo, je vois :)
Ok, donc ici on peut utiliser ton helper select_multi
, si je comprends bien.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tout à fait !
@kemenaran c'est good j'ai ajouté le |
app/controllers/new_administrateur/experts_procedures_controller.rb
Outdated
Show resolved
Hide resolved
spec/controllers/new_administrateur/experts_procedures_controller_spec.rb
Outdated
Show resolved
Hide resolved
ac85441
to
8bab293
Compare
#5813
La vue administrateur lorsque le feature flag est activé
Flag désactivé : l'instructeur peut demander l'avis aux experts de son choix
Flag activé : l'instructeur peut demander l'avis uniquement aux experts de la liste de l'administrateur
Important : Si le flag est activé, un expert ne peut pas demander l'avis d'un autre expert.