Skip to content

GPS: Permettre aux orienteurs d’accéder à GPS sur leur dashboard [GEN-2226]#5100

Merged
tonial merged 1 commit into
masterfrom
alaurent/gps
Nov 18, 2024
Merged

GPS: Permettre aux orienteurs d’accéder à GPS sur leur dashboard [GEN-2226]#5100
tonial merged 1 commit into
masterfrom
alaurent/gps

Conversation

@tonial

@tonial tonial commented Nov 16, 2024

Copy link
Copy Markdown
Contributor

🤔 Pourquoi ?

🍰 Comment ?

Décrivez en quelques mots la solution retenue et mise en oeuvre, les difficultés ou problèmes rencontrés. Attirez l'attention sur les décisions d'architecture ou de conception importantes.

🚨 À vérifier

  • Mettre à jour le CHANGELOG_breaking_changes.md ?

🏝️ Comment tester

Les instructions pour reproduire le problème, les profils de test, le parcours spécifique à utiliser, etc. Si vous disposez d'une recette jetable, mettre l'URL pour tester dans cette partie.

💻 Captures d'écran

@tonial tonial added 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC modifié Modifié dans le changelog. labels Nov 16, 2024
@tonial tonial self-assigned this Nov 16, 2024
@notion-workspace

Copy link
Copy Markdown

@tonial
tonial force-pushed the alaurent/gps branch 3 times, most recently from c68b2a0 to 1ebf995 Compare November 16, 2024 06:53
@tonial tonial added 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC and removed 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC labels Nov 16, 2024
@github-actions

Copy link
Copy Markdown

🥁 La recette jetable est prête ! 👉 Je veux tester cette PR !

@louije

louije commented Nov 16, 2024

Copy link
Copy Markdown
Member

Typo: y’a alternativement la graphie advenced et advanced qui est utilisée

Comment thread tests/gps/test_views.py Outdated
if access is None:
assertRedirects(response, reverse("dashboard:index"))
elif access == "partial":
assertNotContains(response, FEATURE_INVITE)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hum, le ticket Notion indique que les orienteurs doivent pouvoir inviter des partenaires via le Tally, non ?
image

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Ah, oui, mince...

@louije louije Nov 18, 2024

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

(Voui mais c'est un flux complexe que personne n'a encore utilisé donc on va sûrement le repenser un peu)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

C'est bon, je l'ai mis en place pour eux quand même :)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🤔 Il est toujours caché dans my_groups et le test indique qu'il ne doit pas être affiché.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Bien vu !
corrigé aussi

@tonial
tonial force-pushed the alaurent/gps branch 2 times, most recently from a2233be to 78f7b67 Compare November 18, 2024 09:03
@tonial
tonial requested a review from celine-m-s November 18, 2024 09:03
@tonial
tonial enabled auto-merge November 18, 2024 09:17
@tonial
tonial added this pull request to the merge queue Nov 18, 2024
Merged via the queue into master with commit 3f7f8ec Nov 18, 2024
@tonial
tonial deleted the alaurent/gps branch November 18, 2024 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC modifié Modifié dans le changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants