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

misc: add site option to avoid godo for text with django templates (#86222) #1074

Merged
fpeters merged 1 commits from wip/86222-feature-flag-godo into main 2024-01-29 14:44:45 +01:00
Conversation 1 Commits 1 Files Changed 4 +74 -6
fpeters commented 2024-01-27 20:21:37 +01:00
Owner
Copy Link
No description provided.
fpeters force-pushed wip/86222-feature-flag-godo from 53a893fac4 to 7d021e5484 2024-01-28 08:38:50 +01:00 Compare
fpeters changed title from WIP: misc: add site option to avoid godo for text with django templates (#86222) to misc: add site option to avoid godo for text with django templates (#86222) 2024-01-28 08:54:39 +01:00
fpeters reviewed 2024-01-28 08:56:57 +01:00
wcs/qommon/form.py Outdated
@ -3977,0 +3977,4 @@
# in site options for this particular usage.
disabled_godo_usage = [
x.strip() for x in get_publisher().get_site_option('disabled-godo-usages').split(',')
]
fpeters commented 2024-01-28 08:56:57 +01:00
Author
Owner
Copy Link

Dans un premier commit j'avais juste mis un feature flag booléen puis je me suis dit qu'à activer godo pour davantage d'actions, je ne voudrais pas que sur une prochaine action il y ait des retours pour dire que ça ne va pas, et que le feature flag désactive tout, donc j'en fais une liste, pour pouvoir désactiver de manière plus sélective s'il fallait.

Dans un premier commit j'avais juste mis un feature flag booléen puis je me suis dit qu'à activer godo pour davantage d'actions, je ne voudrais pas que sur une prochaine action il y ait des retours pour dire que ça ne va pas, et que le feature flag désactive tout, donc j'en fais une liste, pour pouvoir désactiver de manière plus sélective s'il fallait.
fpeters changed title from misc: add site option to avoid godo for text with django templates (#86222) to WIP: misc: add site option to avoid godo for text with django templates (#86222) 2024-01-28 11:17:29 +01:00
fpeters commented 2024-01-28 11:21:43 +01:00
Author
Owner
Copy Link

Reprendre pour être plus fin encore, permettre selon les usages différents modes, avec les possibilités qui pourraient être :

  • auto-ckeditor : godo, ckeditor si godo pas ok (balisage HTML ou django pas géré)
  • auto-ckeditor-textarea : godo, ckeditor si godo pas ok pour le balisage HTML, textarea si syntaxe django utilisée
  • auto-textarea : godo, textarea bête si godo pas ok
  • godo : tout le temps godo
  • ckeditor : tout le temps ckeditor
  • textarea : tout le temps textrea
Reprendre pour être plus fin encore, permettre selon les usages différents modes, avec les possibilités qui pourraient être : * auto-ckeditor : godo, ckeditor si godo pas ok (balisage HTML ou django pas géré) * auto-ckeditor-textarea : godo, ckeditor si godo pas ok pour le balisage HTML, textarea si syntaxe django utilisée * auto-textarea : godo, textarea bête si godo pas ok * godo : tout le temps godo * ckeditor : tout le temps ckeditor * textarea : tout le temps textrea
fpeters force-pushed wip/86222-feature-flag-godo from 7d021e5484 to c1efe7dcbf 2024-01-28 11:49:33 +01:00 Compare
fpeters changed title from WIP: misc: add site option to avoid godo for text with django templates (#86222) to misc: add site option to avoid godo for text with django templates (#86222) 2024-01-28 18:27:50 +01:00
csechet approved these changes 2024-01-29 13:59:58 +01:00
fpeters merged commit 61fcfee1f5 into main 2024-01-29 14:44:45 +01:00
fpeters deleted branch wip/86222-feature-flag-godo 2024-01-29 14:44:45 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
csechet
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#1074
No description provided.
Delete Branch "wip/86222-feature-flag-godo"

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?