Skip to content

  • Projects
  • Groups
  • Snippets
  • Help
  • This project
    • Loading...
  • Sign in / Register
K
kohinos-tav
  • Overview
    • Overview
    • Details
    • Activity
    • Cycle Analytics
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Charts
  • Issues 0
    • Issues 0
    • List
    • Board
    • Labels
    • Milestones
  • Merge Requests 1
    • Merge Requests 1
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
    • Charts
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Members
    • Members
  • Collapse sidebar
  • Activity
  • Graph
  • Charts
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • agplv3
  • kohinos-tav
  • Merge Requests
  • !61

Merged
Opened Mar 01, 2024 by Damien Moulard@DamienM 
  • Report abuse
Report abuse

dont display presta self eval form if not submitted, in public page

  • Discussion 3
  • Commits 1
  • Changes 2
{{ resolvedDiscussionCount }}/{{ discussionCount }} {{ resolvedCountText }} resolved
  • Yvon Kerdoncuff
    @Yvon started a discussion Mar 02, 2024
    Last updated by Damien Moulard Mar 04, 2024
    • Yvon Kerdoncuff @Yvon commented Mar 02, 2024
      Master

      Pour moi ce n'est pas un bug car il ne se produit que dans la situation où le prestataire est activé et le quiz non soumis, ce qui n'est pas une situation prévue dans le flow d'inscription (l'admin ne doit pas activer le point de vente avant la soumission du formulaire).

      La désactivation du prestataire permet en effet d'empêcher l'accès à la vitrine du prestataire en question, et donc au questionnaire.

      Edited Mar 02, 2024 by Yvon Kerdoncuff
      Pour moi ce n'est pas un bug car il ne se produit que dans la situation où le prestataire est activé et le quiz non soumis, ce qui n'est pas une situation prévue dans le flow d'inscription (l'admin ne doit pas activer le point de vente avant la soumission du formulaire). La désactivation du prestataire permet en effet d'empêcher l'accès à la vitrine du prestataire en question, et donc au questionnaire.
    • Yvon Kerdoncuff @Yvon commented Mar 02, 2024
      Master

      Idéalement, il faudrait empêcher l'activation d'un compte prestataire dont le quiz n'est pas soumis, mais je pense que c'est pénible à faire.

      Et si on considère que le risque d'activation anticipée existe, il faut se poser la question du use case.

      Edited Mar 02, 2024 by Yvon Kerdoncuff
      Idéalement, il faudrait empêcher l'activation d'un compte prestataire dont le quiz n'est pas soumis, mais je pense que c'est pénible à faire. Et si on considère que le risque d'activation anticipée existe, il faut se poser la question du use case.
    • Damien Moulard @DamienM commented Mar 04, 2024
      Master

      D'accord, cela dit la modif de code proposée rajoute une sécurité sans risque de bug supplémantaire, ça ne coute rien de la garder à mon avis.

      D'accord, cela dit la modif de code proposée rajoute une sécurité sans risque de bug supplémantaire, ça ne coute rien de la garder à mon avis.
    Please register or sign in to reply
  • Yvon Kerdoncuff @Yvon

    mentioned in commit 6e0157a8

    Mar 05, 2024

    mentioned in commit 6e0157a8

    mentioned in commit 6e0157a80d3e958ce7cac04bb23f67bad4bef8ea
    Toggle commit list
  • Yvon Kerdoncuff @Yvon

    merged

    Mar 05, 2024

    merged

    merged
    Toggle commit list
  • Write
  • Preview
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or sign in to comment
Assignee
No assignee
Assign to
None
Milestone
None
Assign milestone
Time tracking
Reference: agplv3/kohinos-tav!61
×

Revert this merge request

This will create a new commit in order to revert the existing changes.

Switch branch
Cancel
A new branch will be created in your fork and a new merge request will be started.
×

Cherry-pick this merge request

Switch branch
Cancel
A new branch will be created in your fork and a new merge request will be started.