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
  • !52

Merged
Opened Feb 15, 2024 by Yvon Kerdoncuff@Yvon 
  • Report abuse
Report abuse

5748 visualiser et evaluer autoevaluation

  • Discussion 8
  • Commits 5
  • Changes 20
{{ resolvedDiscussionCount }}/{{ discussionCount }} {{ resolvedCountText }} resolved
  • Damien Moulard
    @DamienM started a discussion Feb 19, 2024
    Resolved by Damien Moulard Feb 20, 2024
    • Damien Moulard @DamienM commented Feb 19, 2024
      Master

      Je rajouterais de la marge entre la section "évaluation globale du pdv" et "évaluation du gestionnaire"

      image

      Je rajouterais de la marge entre la section "évaluation globale du pdv" et "évaluation du gestionnaire" ![image](/uploads/f789efea06dd5debc33f1090b1746d9c/image.png)
    • Yvon Kerdoncuff @Yvon commented Feb 20, 2024
      Master

      ok

      Edited Feb 20, 2024 by Yvon Kerdoncuff
      ok
    Please register or sign in to reply
  • Damien Moulard
    @DamienM started a discussion Feb 19, 2024
    Resolved by Damien Moulard Feb 20, 2024
    • Damien Moulard @DamienM commented Feb 19, 2024
      Master

      Je pense qu'il peut être intéressant de rajouter l'information de si le prestataire est producteur ou distributeur.

      Même si en soit ça peut être déduit par le formulaire, je pense que ça facilitera la vie des gestionnaires si l'info est rapidement visible en haut de la page. Qu'en penses-tu ?

      Je pense qu'il peut être intéressant de rajouter l'information de si le prestataire est producteur ou distributeur. Même si en soit ça peut être déduit par le formulaire, je pense que ça facilitera la vie des gestionnaires si l'info est rapidement visible en haut de la page. Qu'en penses-tu ?
    • Yvon Kerdoncuff @Yvon commented Feb 20, 2024
      Master

      carrément, j'en profite pour afficher la raison du prestataire

      carrément, j'en profite pour afficher la raison du prestataire
    Please register or sign in to reply
  • Damien Moulard
    @DamienM started a discussion Feb 19, 2024
    Resolved by Damien Moulard Feb 20, 2024
    • Damien Moulard @DamienM commented Feb 19, 2024
      Master

      En l'état, un gestionnaire de comptoir a accès au bouton "Revue QCM".

      Le bouton ne devrait être accessibles aux rôles admin et gestionnaires de groupe.

      Edited Feb 19, 2024 by Damien Moulard
      En l'état, un gestionnaire de comptoir a accès au bouton "Revue QCM". Le bouton ne devrait être accessibles aux rôles admin et gestionnaires de groupe.
    • Yvon Kerdoncuff @Yvon commented Feb 20, 2024
      Master

      bien vu ! du coup j'ai restreint à role_gestion_groupe je pensais que ça suffirait mais non j'ai dû permettre expicitement role_super_admin également

      Observation : le role administrateur du siège (qui n'a pas l'air d'être utilisé en pratique...) ne peut pas dérouler le menu Prestataire dans l'admin

      bien vu ! du coup j'ai restreint à role_gestion_groupe je pensais que ça suffirait mais non j'ai dû permettre expicitement role_super_admin également Observation : le role administrateur du siège (qui n'a pas l'air d'être utilisé en pratique...) ne peut pas dérouler le menu Prestataire dans l'admin
    • Yvon Kerdoncuff @Yvon commented Feb 20, 2024
      Master

      Il manquerait aussi peut-être "ROLE_ADMIN" mais le donner donne aussi le droit au comptoir, redacteur, etc etc comme le précise la hiérarchie des rôles dans packages/security.yaml. Les exclure explicitement deviendrait verbeux !

      Il manquerait aussi peut-être "ROLE_ADMIN" mais le donner donne aussi le droit au comptoir, redacteur, etc etc comme le précise la hiérarchie des rôles dans packages/security.yaml. Les exclure explicitement deviendrait verbeux !
    • Damien Moulard @DamienM commented Feb 20, 2024
      Master

      Oui je crois que "ROLE_ADMIN" c'est tous ceux qui ont accès au panel d'admin

      Oui je crois que "ROLE_ADMIN" c'est tous ceux qui ont accès au panel d'admin
    Please register or sign in to reply
  • Yvon Kerdoncuff @Yvon

    added 3 commits

    • b70a9b81 - margin between presta global eval area and review area
    • 481ebda9 - improve qcm review titles
    • 17ac2c9f - restrict Revue QCM btn visibility to role_gestion_groupe and role_super_admin

    Compare with previous version

    Feb 20, 2024

    added 3 commits

    • b70a9b81 - margin between presta global eval area and review area
    • 481ebda9 - improve qcm review titles
    • 17ac2c9f - restrict Revue QCM btn visibility to role_gestion_groupe and role_super_admin

    Compare with previous version

    added 3 commits <ul><li>b70a9b81 - margin between presta global eval area and review area</li><li>481ebda9 - improve qcm review titles</li><li>17ac2c9f - restrict Revue QCM btn visibility to role_gestion_groupe and role_super_admin</li></ul> [Compare with previous version](https://gl.cooperatic.fr/cooperatic/kohinos-tav/merge_requests/52/diffs?diff_id=2772&start_sha=96a02832b19c627e5ab0c39b8074bf958f4d7c3b)
    Toggle commit list
  • Damien Moulard @DamienM

    resolved all discussions

    Feb 20, 2024

    resolved all discussions

    resolved all discussions
    Toggle commit list
  • Damien Moulard @DamienM

    mentioned in commit 6921115c

    Feb 20, 2024

    mentioned in commit 6921115c

    mentioned in commit 6921115c634dfebccb9066003f726e46985f6851
    Toggle commit list
  • Damien Moulard @DamienM

    merged

    Feb 20, 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!52
×

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.