entrouvert
/
passerelle
15
0
Fork 0
Code Issues Pull Requests 13 Releases Activity

toulouse_foederis: simplify document model (#76680) #212

Merged
bdauvergne merged 1 commits from wip/76680-toulouse-foederis-erreur-sur-int into main 2023-04-17 18:23:55 +02:00
Conversation 1 Commits 1 Files Changed 5 +76 -4
bdauvergne commented 2023-04-16 08:51:43 +02:00
Owner
Copy Link
  • no null, only blank and default values.
  • no limited charfield, all textfield.
- no null, only blank and default values. - no limited charfield, all textfield.
bdauvergne changed title from WIP: toulouse_foderis: simplify document model (#76680) to toulouse_foderis: simplify document model (#76680) 2023-04-16 09:10:24 +02:00
vdeniaud reviewed 2023-04-17 16:01:19 +02:00
passerelle/contrib/toulouse_foederis/migrations/0002_no_null_no_charfield.py Outdated
@ -0,0 +5,4 @@
def set_default(apps, schema_editor):
# prevent postgresql error "pending event triggers" when altering the nullable status
schema_editor.execute('SET CONSTRAINTS ALL IMMEDIATE')
vdeniaud commented 2023-04-17 16:01:19 +02:00
Owner
Copy Link

Pas à l'aise avec ça, perso dans la doc je lis https://docs.djangoproject.com/en/3.2/ref/migration-operations/#runpython :

you should avoid combining schema changes and RunPython operations in the same migration

Sous-entendu la bonne pratique est de séparer en deux fichiers, pas exécuter du SQL (c'est aussi ce qu'on fait ailleurs à chaque fois que le problème se pose).

Pas à l'aise avec ça, perso dans la doc je lis https://docs.djangoproject.com/en/3.2/ref/migration-operations/#runpython : > you should avoid combining schema changes and RunPython operations in the same migration Sous-entendu la bonne pratique est de séparer en deux fichiers, pas exécuter du SQL (c'est aussi ce qu'on fait ailleurs à chaque fois que le problème se pose).
vdeniaud approved these changes 2023-04-17 16:01:32 +02:00
nroche reviewed 2023-04-17 16:39:06 +02:00
passerelle/contrib/toulouse_foederis/models.py Outdated
@ -357,4 +357,1 @@
)
external_id = models.CharField(_('Key'), max_length=64, unique=True)
text = models.CharField(_('Text'), max_length=64, null=True)
data = JSONField(_('Data'), null=True)
nroche commented 2023-04-17 11:52:29 +02:00
Owner
Copy Link

Question, c'est quoi qui est corrigé ici ?
La seule référence que j'ai sur dafault=dict c'est https://dev.entrouvert.org/issues/42312#note-6
(je demande parce que ça n'a pas posé de problème jusque là,
mais que si c'est vraiment une erreur à corriger, alors je devrais appliquer la même correction
sur le connecteur toulouse-maelis)

Question, c'est quoi qui est corrigé ici ? La seule référence que j'ai sur dafault=dict c'est https://dev.entrouvert.org/issues/42312#note-6 (je demande parce que ça n'a pas posé de problème jusque là, mais que si c'est vraiment une erreur à corriger, alors je devrais appliquer la même correction sur le connecteur toulouse-maelis)
bdauvergne commented 2023-04-17 18:13:48 +02:00
Author
Owner
Copy Link

Non y a pas d'erreur, mais une base de donn

Non y a pas d'erreur, mais une base de donn
nroche commented 2023-04-18 08:06:07 +02:00
Owner
Copy Link

C'est étrange, tes messages sont coupés au premier accent rencontré, mais pas les miens.

C'est étrange, tes messages sont coupés au premier accent rencontré, mais pas les miens.
fpeters commented 2023-04-18 08:13:37 +02:00
Owner
Copy Link

C'est étrange, tes messages sont coupés au premier accent rencontré, mais pas les miens.

Oui problème connu, touches mortes, Agate a proposé uen solution en opt-in pour ne pas avoir le problème (tout en n'éliminant pas l'édition riche), c'est au bas de https://dev.entrouvert.org/issues/73988#note-4 :

C'est plus complexe, mais ça a l'air de marcher partout. C'est de l'optin, il faut lancer localStorage.setItem("useTextarea", 'true') au moins une fois dans sa console navigateur pour que ça soit affiché, ça permet de continuer à utiliser l'éditeur plus complet si on le souhaite.

Je viens de créer un titre Gitea ici https://dev.entrouvert.org/projects/prod-eo/wiki/HowDoWeDoGit#Gitea et j'y ai copié/collé l'info.

> C'est étrange, tes messages sont coupés au premier accent rencontré, mais pas les miens. Oui problème connu, touches mortes, Agate a proposé uen solution en opt-in pour ne pas avoir le problème (tout en n'éliminant pas l'édition riche), c'est au bas de https://dev.entrouvert.org/issues/73988#note-4 : > C'est plus complexe, mais ça a l'air de marcher partout. C'est de l'optin, il faut lancer localStorage.setItem("useTextarea", 'true') au moins une fois dans sa console navigateur pour que ça soit affiché, ça permet de continuer à utiliser l'éditeur plus complet si on le souhaite. Je viens de créer un titre Gitea ici https://dev.entrouvert.org/projects/prod-eo/wiki/HowDoWeDoGit#Gitea et j'y ai copié/collé l'info.
bdauvergne force-pushed wip/76680-toulouse-foederis-erreur-sur-int from 0f98bc7a0a to cf808b1d5c 2023-04-17 17:41:05 +02:00 Compare
fpeters commented 2023-04-17 17:49:41 +02:00
Owner
Copy Link

détail dans le message de commit (et l'intitulé de la PR), c'est foderis pas foederis.

détail dans le message de commit (et l'intitulé de la PR), c'est foderis pas foederis.
bdauvergne force-pushed wip/76680-toulouse-foederis-erreur-sur-int from cf808b1d5c to 2d518f5f06 2023-04-17 18:16:16 +02:00 Compare
bdauvergne changed title from toulouse_foderis: simplify document model (#76680) to toulouse_foederis: simplify document model (#76680) 2023-04-17 18:16:26 +02:00
bdauvergne merged commit 5314e91ace into main 2023-04-17 18:23:55 +02:00
bdauvergne deleted branch wip/76680-toulouse-foederis-erreur-sur-int 2023-04-17 18:23:55 +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
4 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/passerelle#212
No description provided.
Delete Branch "wip/76680-toulouse-foederis-erreur-sur-int"

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?