entrouvert
/
chrono
14
1
Fork 0
Code Issues Pull Requests 7 Releases Wiki Activity

agendas: use URLField for event url (#89447) #245

Merged
tnoel merged 1 commits from wip/89447-event-url-urlfield into main 2024-04-16 18:01:42 +02:00
Conversation 0 Commits 1 Files Changed 3 +20 -12
tnoel commented 2024-04-12 00:36:03 +02:00
Owner
Copy Link

Très simplement, imposer un format via URLField (qui demande https, http, ftp, ftps). La migration ne modifie pas l'existant, ie on considère que les URL actuelles sont OK.

Très simplement, imposer un format via URLField (qui demande https, http, ftp, ftps). La migration ne modifie pas l'existant, ie on considère que les URL actuelles sont OK.
tnoel added 1 commit 2024-04-12 00:36:03 +02:00
gitea/chrono/pipeline/head There was a failure building this commit Details
3dd007c23b agendas: use URLField for event url (#89447)
tnoel changed title from agendas: use URLField for event url (#89447) to WIP: agendas: use URLField for event url (#89447) 2024-04-12 00:36:29 +02:00
tnoel force-pushed wip/89447-event-url-urlfield from 3dd007c23b to 8ce028a725 2024-04-12 01:00:58 +02:00 Compare
tnoel force-pushed wip/89447-event-url-urlfield from 8ce028a725 to 6de3779c90 2024-04-12 01:05:32 +02:00 Compare
tnoel changed title from WIP: agendas: use URLField for event url (#89447) to agendas: use URLField for event url (#89447) 2024-04-12 01:08:19 +02:00
vdeniaud approved these changes 2024-04-16 09:49:04 +02:00
Dismissed
chrono/agendas/migrations/0172_alter_event_url.py Outdated
@ -0,0 +3,4 @@
from django.db import migrations, models
class Migration(migrations.Migration):
vdeniaud commented 2024-04-16 09:48:05 +02:00
Owner
Copy Link

À noter qu'on pourrait sans problème faire sans cette migration, URLField ça met juste un validateur python et éventuellement ça ajoute un max_length, mais pas ici puisqu'il est spécifié explicitement : donc la migration ne générera pas de SQL.

À noter qu'on pourrait sans problème faire sans cette migration, URLField ça met juste un validateur python et éventuellement ça ajoute un max_length, mais pas ici puisqu'il est spécifié explicitement : donc la migration ne générera pas de SQL.
tnoel commented 2024-04-16 17:27:34 +02:00
Author
Owner
Copy Link

Éviter une migration c'est toujours bien, même quand ça ne génère aucun SQL, on gagne du temps. Je reporte l'affaire sur la 0036

Éviter une migration c'est toujours bien, même quand ça ne génère aucun SQL, on gagne du temps. Je reporte l'affaire sur la 0036
tnoel marked this conversation as resolved
tnoel force-pushed wip/89447-event-url-urlfield from 6de3779c90 to 6ac6034605 2024-04-16 17:26:45 +02:00 Compare
tnoel requested review from vdeniaud 2024-04-16 17:29:16 +02:00
vdeniaud approved these changes 2024-04-16 17:50:58 +02:00
tnoel merged commit 6ee8fbf78d into main 2024-04-16 18:01:42 +02:00
tnoel deleted branch wip/89447-event-url-urlfield 2024-04-16 18:01:42 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
vdeniaud
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/chrono#245
No description provided.
Delete Branch "wip/89447-event-url-urlfield"

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?