entrouvert
/
authentic
16
0
Fork 0
Code Issues Pull Requests 35 Releases Activity

Recherche sur les rôles insensibles aux accents (#87906) #289

Merged
bdauvergne merged 1 commits from wip/87906-search-role-unaccent into main 2024-04-15 15:19:04 +02:00
Conversation 2 Commits 1 Files Changed 5 +59 -3
bdauvergne commented 2024-04-03 12:39:26 +02:00
Owner
Copy Link
No description provided.
bdauvergne added 2 commits 2024-04-03 12:39:26 +02:00
gitea/authentic/pipeline/head There was a failure building this commit Details
d79547b143 wip
bdauvergne force-pushed wip/87906-search-role-unaccent from d79547b143 to f73fd9c4a2 2024-04-03 12:45:10 +02:00 Compare
bdauvergne force-pushed wip/87906-search-role-unaccent from f73fd9c4a2 to 188b8d916c 2024-04-03 12:53:49 +02:00 Compare
bdauvergne changed title from WIP: Recherche sur les rôles insensibles aux accents (#87906) to Recherche sur les rôles insensibles aux accents (#87906) 2024-04-03 12:54:00 +02:00
tnoel commented 2024-04-03 14:41:58 +02:00
Owner
Copy Link

Si je comprends bien, dans cette nouvelle version on joue avec public.immutable_accent déclarée dans src/authentic2/migrations/0028_trigram_unaccent_index.py

J'ai bon ?

Si je comprends bien, dans cette nouvelle version on joue avec public.immutable_accent déclarée dans src/authentic2/migrations/0028_trigram_unaccent_index.py J'ai bon ?
bdauvergne commented 2024-04-03 14:47:35 +02:00
Author
Owner
Copy Link

Si je comprends bien, dans cette nouvelle version on joue avec public.immutable_accent déclarée dans src/authentic2/migrations/0028_trigram_unaccent_index.py

J'ai bon ?

Oui le souci ici c'est que autant pendant les requêtes django-tenants-schemas met public dans le search_path pendant les migrations il ne le met pas, donc le code fonctionne sans django-tenant-schemas (dans les tests) mais pas avec django-tenant-schemas pendant les migrations (et si jamais la migration était passée ça aurait marché dans uwsgi). Le mieux c'est de toujours mettre le préfixe de schéma si c'est dans le schéma public.

> Si je comprends bien, dans cette nouvelle version on joue avec public.immutable_accent déclarée dans src/authentic2/migrations/0028_trigram_unaccent_index.py > > J'ai bon ? Oui le souci ici c'est que autant pendant les requêtes django-tenants-schemas met public dans le search_path pendant les migrations il ne le met pas, donc le code fonctionne sans django-tenant-schemas (dans les tests) mais pas avec django-tenant-schemas pendant les migrations (et si jamais la migration était passée ça aurait marché dans uwsgi). Le mieux c'est de toujours mettre le préfixe de schéma si c'est dans le schéma public.
bdauvergne requested review from tnoel 2024-04-08 16:43:24 +02:00
tnoel approved these changes 2024-04-08 16:59:28 +02:00
tnoel left a comment
Owner
Copy Link

Ça marche ainsi, merci ; à pousser dans le cycle prochain cependant (à partir du 12 avril)

Ça marche ainsi, merci ; à pousser dans le cycle prochain cependant (à partir du 12 avril)
bdauvergne merged commit 2b3d04a6d1 into main 2024-04-15 15:19:04 +02:00
bdauvergne deleted branch wip/87906-search-role-unaccent 2024-04-15 15:19:04 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
tnoel
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/authentic#289
No description provided.
Delete Branch "wip/87906-search-role-unaccent"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?