search

osm-fr / sotmfr-website

Code du site web de l'événement State of the Map
https://sotm2024.openstreetmap.fr/
7 stars 10 forks source link

Gestions des droits #112

Closed vinber closed 1 year ago

vinber commented 1 year ago

@JackNUMBER @marcelezd @ThomasG77 @tykayn

vous avez meilleure connaissance que moi des droits et de ce qu'il vous semble préférable d'avoir sur ce dépot, n'hésitez pas à me dire !

ce que je trouve "bien" c'est le fait d'avoir quand même 2 étapes, celui qui fait, celui qui merge et déploie mais peut-être n'est-ce pas nécessaire au regard de l'enjeu du site (somme tout enjeu pas hyper grave ...) !

@JackNUMBER et @marcelezd j'ai essayé de vous ajouter à l'équipe, par exemple pour que vous puissiez vous assigner des issues et/ou autre droit mais pas réussi :(

@tykain tu as les droits d'écriture

JackNUMBER commented 1 year ago

On pourrait rendre l'approbation de PR obligatoire pour merger une PR (doc). Il me semble qu'il est possible d'attribuer des droits pour que certaines personnes puissent bypasser cela.

~Et mettre en place un déploiement automatique au merge (doc) ainsi seul le système connaitrait les secrets d'accès au serveur. Je peux aider à la mise en place (sans avoir accès aux secrets car c'est l'admin qui les aura définis).~ EDIT : Le déploiement actuel est automatisé.

Tout le monde pourrait avoir accès aux 2 étapes dont tu parles mais pas possible de valider son propre code et donc de déployer.

vinber commented 1 year ago

ok, l'idée c'est d'avoir

il va falloir me guider hein ! je veux bien donner des droits à tout le monde, cela me semblerait même assez cohérent de donnéer tous les droits à tout le monde mais je ne suis pas sur que cela marche comme cela :)

vinber commented 1 year ago

@JackNUMBER @marcelezd je ne sais pas comment changer vos droits ... vous n'apparraissez pas dans l'équipe, je ne trouve donc pas le moyen de vous changer les droits ... je creuse @ThomasG77 si tu as une idée !

cela permettrait de faire passer les PR plus rapidement

marcelezd commented 1 year ago

la doc pour ajouter quelqu'un est ici avec sur cette page les actions possibles en fonction des différents rôles

vinber commented 1 year ago

Merci, normalement @JackNUMBER et @marcelezd vous avez des droits plus étendus

vinber commented 1 year ago

Pour la petite histoire, je ne voyais pas le add (ma faute), je ne cherchais que dans l'équipe..

vinber commented 1 year ago

On reste sur la proposition je ne merge PAS ma PR c'est bien cela. Pour l'instant c'est pas technique, seulement par accord

JackNUMBER commented 1 year ago

On peut clore cette issue ?