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

settings: mark options set by hobo as readonly (#44756) #844

Merged
fpeters merged 2 commits from wip/44756-hobo-readonly-settings into main 2023-11-27 16:28:32 +01:00
Conversation 0 Commits 2 Files Changed 5 +101 -32
fpeters commented 2023-11-18 10:29:25 +01:00
Owner
Copy Link
No description provided.
fpeters force-pushed wip/44756-hobo-readonly-settings from 2a70e912ff to 26a78a9a4a 2023-11-18 11:06:42 +01:00 Compare
fpeters force-pushed wip/44756-hobo-readonly-settings from 26a78a9a4a to 00440c0100 2023-11-18 11:29:42 +01:00 Compare
fpeters reviewed 2023-11-18 12:03:24 +01:00
fpeters left a comment
Author
Owner
Copy Link

Avec ça + ce qui peut déjà se cacher/désactiver, on arrive je pense à un /backoffice/settings/ qui pourrait être accessible.

Avec ça + ce qui peut déjà se cacher/désactiver, on arrive je pense à un /backoffice/settings/ qui pourrait être accessible.
tests/admin_pages/test_settings.py
@ -85,1 +85,3 @@
pub.site_options.set('options', 'settings-disabled-screens', 'identification, import-export, geolocation')
pub.site_options.set(
'options', 'settings-disabled-screens', 'identification, import-export, geolocation, smtp'
)
fpeters commented 2023-11-18 12:01:27 +01:00
Author
Owner
Copy Link

Un premier commit qui ajoute une valeur "smtp" pour settings-disabled-screens, pour ne pas afficher les options smtp_server, smtp_login, smtp_password.

Un premier commit qui ajoute une valeur "smtp" pour settings-disabled-screens, pour ne pas afficher les options smtp_server, smtp_login, smtp_password.
tests/admin_pages/test_settings.py Outdated
@ -1039,0 +1053,4 @@
try:
with open(hobo_json_path, 'w'):
resp = app.get('/backoffice/settings/sitename')
assert resp.pyquery('#form_sitename').attr.readonly
fpeters commented 2023-11-18 12:02:30 +01:00
Author
Owner
Copy Link

Et un deuxième commit qui en présence d'un déploiement via hobo (via la présence du fichier hobo.json), marque en lecture seule les paramètres qui en viennent. (ici sitename, mais aussi les paramètres de footer d'email, de connecteur sms, etc.).

Et un deuxième commit qui en présence d'un déploiement via hobo (via la présence du fichier hobo.json), marque en lecture seule les paramètres qui en viennent. (ici sitename, mais aussi les paramètres de footer d'email, de connecteur sms, etc.).
fpeters changed title from WIP: settings: mark options set by hobo as readonly (#44756) to settings: mark options set by hobo as readonly (#44756) 2023-11-18 12:03:29 +01:00
tnoel reviewed 2023-11-18 12:09:47 +01:00
wcs/qommon/admin/cfg.py Outdated
@ -20,0 +23,4 @@
def hobo_kwargs():
if os.path.exists(os.path.join(get_publisher().tenant.directory, 'hobo.json')):
return {'readonly': True, 'attrs': {'title': _('This setting is locked-down by deployment.')}}
tnoel commented 2023-11-18 12:09:47 +01:00
Owner
Copy Link

Plutôt ajouter un "hint" que remplacer le "title" ?

Plutôt ajouter un "hint" que remplacer le "title" ?
fpeters commented 2023-11-18 13:33:15 +01:00
Author
Owner
Copy Link

Oui ok, en écrasant la remarque si jamais elle existait (comme le champ ne peut pas être modifié, pas bien grave de la perdre, je pense).

Oui ok, en écrasant la remarque si jamais elle existait (comme le champ ne peut pas être modifié, pas bien grave de la perdre, je pense).
fpeters force-pushed wip/44756-hobo-readonly-settings from 00440c0100 to f93ba57658 2023-11-18 12:25:13 +01:00 Compare
tnoel approved these changes 2023-11-27 16:27:45 +01:00
fpeters merged commit d546f91f33 into main 2023-11-27 16:28:32 +01:00
fpeters deleted branch wip/44756-hobo-readonly-settings 2023-11-27 16:28:32 +01: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/wcs#844
No description provided.
Delete Branch "wip/44756-hobo-readonly-settings"

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?