entrouvert/wcs
entrouvert
/
wcs
14
1
Fork 0
Code Issues Pull Requests 31 Releases Activity

ajustements à misc.simplify (#21227) #855

Merged
fpeters merged 4 commits from wip/21227-misc-simplify-underscores into main 2023-11-24 09:13:15 +01:00
Conversation 0 Commits 4 Files Changed 5 +22 -11
fpeters commented 2023-11-21 15:52:37 +01:00
Owner
Copy Link

De base le ticket c'était pour traiter uniformémment les tirets et les underscores, ne pas se trouver avec une chaine mixant les deux, à ça j'ajoute un .strip() pour ne pas faire commencer/terminer les chaines par le caractère de substitution (c'est quelque chose qui a été modifié dans le slugify de django aussi, "Fixed #28694 -- Made django.utils.text.slugify() strip dashes and underscores."); puis un commit pour lancer la réindexation des formdata pour prendre en compte le changement, puis un commit pour retirer un petit hack que j'avais mis dans la création du slug du nom du trigger.

De base le ticket c'était pour traiter uniformémment les tirets et les underscores, ne pas se trouver avec une chaine mixant les deux, à ça j'ajoute un .strip() pour ne pas faire commencer/terminer les chaines par le caractère de substitution (c'est quelque chose qui a été modifié dans le slugify de django aussi, "Fixed #28694 -- Made django.utils.text.slugify() strip dashes and underscores."); puis un commit pour lancer la réindexation des formdata pour prendre en compte le changement, puis un commit pour retirer un petit hack que j'avais mis dans la création du slug du nom du trigger.
fpeters force-pushed wip/21227-misc-simplify-underscores from 01a4baf332 to c0e8e8a4f9 2023-11-21 16:30:51 +01:00 Compare
fpeters changed title from WIP: ajustements à misc.simplify (#21227) to ajustements à misc.simplify (#21227) 2023-11-21 17:02:42 +01:00
fpeters reviewed 2023-11-21 17:03:17 +01:00
tests/test_datasource_chrono.py
@ -297,3 +297,3 @@
datasource2 = NamedDataSource.get(4 + 2)
assert datasource1.name == 'Events A'
assert datasource1.slug == 'chrono_ds_sluga'
assert datasource1.slug == 'chrono_ds_slug_a'
fpeters commented 2023-11-21 17:03:17 +01:00
Author
Owner
Copy Link

Il peut y avoir changement sur les sources de données issues de chrono mais la mise à jour se base sur l'URL, pas sur le slug, donc pas de problème.

Il peut y avoir changement sur les sources de données issues de chrono mais la mise à jour se base sur l'URL, pas sur le slug, donc pas de problème.
lguerin approved these changes 2023-11-23 09:23:30 +01:00
fpeters merged commit 56ae1fa723 into main 2023-11-24 09:13:15 +01:00
fpeters deleted branch wip/21227-misc-simplify-underscores 2023-11-24 09:13:15 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
lguerin
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/wcs#855
No description provided.
Delete Branch "wip/21227-misc-simplify-underscores"

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?