Skip to content

Conversation

@rsebille
Copy link
Contributor

🤔 Pourquoi ?

Éviter une erreur évitable
image

🏝️ Comment tester

  • Se connecter avec une EA/EATT
  • Aller dans une des pages du bloc "Organisation"

💻 Captures d'écran

image

@rsebille rsebille added the modifié Modifié dans le changelog. label Aug 26, 2024
@rsebille rsebille self-assigned this Aug 26, 2024
@rsebille rsebille force-pushed the rsebille/fix-ea-af branch from a633c27 to b7a411c Compare August 26, 2024 15:47
with self.assertNumQueries(16):
# 8. SELECT EXISTS users_user (convention_can_be_accessed_by())
# 9. SELECT companies_siaeconvention (menu checks for financial annexes)
# 10. SELECT EXISTS users_user (menu checks for active admin)
Copy link
Member

Choose a reason for hiding this comment

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

Vivement assertNumQueriesSnapshot 😁

Suggested change
# 10. SELECT EXISTS users_user (menu checks for active admin)
# 10. SELECT EXISTS users_user (menu checks for active admin)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oui ! 👀 @xavfernandez

@rsebille rsebille force-pushed the rsebille/fix-ea-af branch from b7a411c to 50d7154 Compare August 27, 2024 07:12
@rsebille rsebille enabled auto-merge August 27, 2024 07:12
@rsebille rsebille added this pull request to the merge queue Aug 27, 2024
Merged via the queue into master with commit b24921a Aug 27, 2024
@rsebille rsebille deleted the rsebille/fix-ea-af branch August 27, 2024 07:29
@francoisfreitag francoisfreitag changed the title Tableau de bord - Organisation : N'afficher le sous-menu des annexes financières que si il est utilisable Tableau de bord - Organisation : N'afficher le sous-menu des annexes financières que s’il est utilisable Aug 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

modifié Modifié dans le changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants