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

inspecteur: faire apparaître les custom views dans l'inspect formdef et carddef et enregistrer un snapshot du formdef/carddef à la modif d'une custom view (#80235) #579

Merged
lguerin merged 4 commits from wip/80235-custom-view-snapshot into main 2023-08-11 11:50:39 +02:00
Conversation 0 Commits 4 Files Changed 9 +214 -35
lguerin commented 2023-08-10 13:31:12 +02:00
Owner
Copy Link
No description provided.
lguerin changed title from inspecteur: faire apparaître les custom views dans l'inspect formdef et carddef et enregistrer un snapshot du formdef/carddef à la modif d'une custom view (#80235) to WIP: inspecteur: faire apparaître les custom views dans l'inspect formdef et carddef et enregistrer un snapshot du formdef/carddef à la modif d'une custom view (#80235) 2023-08-10 13:31:17 +02:00
lguerin changed title from WIP: inspecteur: faire apparaître les custom views dans l'inspect formdef et carddef et enregistrer un snapshot du formdef/carddef à la modif d'une custom view (#80235) to inspecteur: faire apparaître les custom views dans l'inspect formdef et carddef et enregistrer un snapshot du formdef/carddef à la modif d'une custom view (#80235) 2023-08-10 14:08:50 +02:00
fpeters requested changes 2023-08-11 08:22:10 +02:00
wcs/templates/wcs/backoffice/formdef-inspect.html Outdated
@ -84,0 +99,4 @@
{% elif custom_view.visibility == 'role' %}
{% trans "to role" %} ({{ custom_view.role|default:'-' }})
{% endif %}</li>
<li><span class='parameter'>{% trans "Default vue" %}{% trans ":" %}</span> {{ custom_view.is_default|yesno }}</li>
fpeters commented 2023-08-11 08:20:11 +02:00
Owner
Copy Link

% trans "Default vue" %}

Default view.

> % trans "Default vue" %} Default view.
lguerin commented 2023-08-11 10:09:17 +02:00
Author
Owner
Copy Link

corrigé

corrigé
wcs/templates/wcs/backoffice/formdef-inspect.html Outdated
@ -84,0 +103,4 @@
<li><span class='parameter'>{% trans "Digest" %}{% trans ":" %}</span> {{ custom_view.digest_template|default:'-' }}</li>
<li><span class='parameter'>{% trans "Columns" %}{% trans ":" %}</span> {{ custom_view.columns|default:'-' }}</li>
<li><span class='parameter'>{% trans "Filters" %}{% trans ":" %}</span> {{ custom_view.filters|default:'-' }}</li>
<li><span class='parameter'>{% trans "Order by" %}{% trans ":" %}</span> {{ custom_view.order_by|default:'-' }}</li>
fpeters commented 2023-08-11 08:21:38 +02:00
Owner
Copy Link

Comme j'ai ajouté le regroupement, il pourrait être repris ici :

{% if custom_view.visibility == 'datasource' %}
<li><span class='parameter'>{% trans "Group by" %}{% trans ":" %}</span> {{ custom_view.group_by|default:'-' }}</li> 
{% endif %}
Comme j'ai ajouté le regroupement, il pourrait être repris ici : ``` {% if custom_view.visibility == 'datasource' %} <li><span class='parameter'>{% trans "Group by" %}{% trans ":" %}</span> {{ custom_view.group_by|default:'-' }}</li> {% endif %} ```
lguerin commented 2023-08-11 10:09:27 +02:00
Author
Owner
Copy Link

ajouté

ajouté
wcs/templates/wcs/backoffice/snapshots.html Outdated
@ -88,4 +86,0 @@
$('input[name="version1"]').on('click', function() {
var next = $(this).parent('tr').next();
$('input[name="version2"]', next).prop('checked', true);
});
fpeters commented 2023-08-11 08:19:55 +02:00
Owner
Copy Link

Cette partie pourrait plutôt être corrigée,

- var next = $(this).parent('tr').next();
+ var next = $(this).parents('tr').next()
Cette partie pourrait plutôt être corrigée, ``` - var next = $(this).parent('tr').next(); + var next = $(this).parents('tr').next() ```
lguerin commented 2023-08-11 10:08:55 +02:00
Author
Owner
Copy Link

Je ne sais pas: ça n'a pas l'air d'avoir manqué, et le système de comparaison de version fait de toute façon l'inversion si version1 > version2 (ou le contraire, je ne sais plus).
Autant laisser l'utilisateur cliquer là où il veut sans que ça change sa sélection ?

Et je n'ai pas repris ce JS dans combo :)

Je ne sais pas: ça n'a pas l'air d'avoir manqué, et le système de comparaison de version fait de toute façon l'inversion si version1 > version2 (ou le contraire, je ne sais plus). Autant laisser l'utilisateur cliquer là où il veut sans que ça change sa sélection ? Et je n'ai pas repris ce JS dans combo :)
lguerin force-pushed wip/80235-custom-view-snapshot from 8d2b60156e to 1787d913d5 2023-08-11 10:09:07 +02:00 Compare
fpeters approved these changes 2023-08-11 11:09:59 +02:00
lguerin force-pushed wip/80235-custom-view-snapshot from 1787d913d5 to ba53226b84 2023-08-11 11:16:41 +02:00 Compare
lguerin merged commit ba53226b84 into main 2023-08-11 11:50:39 +02:00
lguerin deleted branch wip/80235-custom-view-snapshot 2023-08-11 11:50:39 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
fpeters
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#579
No description provided.
Delete Branch "wip/80235-custom-view-snapshot"

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?