entrouvert
/
combo
16
1
Fork 0
Code Issues Pull Requests 10 Releases Wiki Activity

Action WCS configurables dans la cellule fiche (#75908) #252

Open
csechet wants to merge 8 commits from wip/75908-wcs-trigger into main
pull from: wip/75908-wcs-trigger
merge into: entrouvert:main
Conversation 0 Commits 8 Files Changed 18 +706 -23
csechet commented 2024-04-16 16:48:41 +02:00
Owner
Copy Link
No description provided.
csechet added 3 commits 2024-04-16 16:48:42 +02:00
gitea/combo/pipeline/head There was a failure building this commit Details
98b4a2c625 cells: add wcs action button cell in frontend card cell (#75908)
csechet force-pushed wip/75908-wcs-trigger from 98b4a2c625 to 11a83f39bf 2024-04-16 16:51:19 +02:00 Compare
csechet force-pushed wip/75908-wcs-trigger from 11a83f39bf to 03a6bfc328 2024-04-16 16:55:21 +02:00 Compare
fpeters requested changes 2024-04-26 09:52:25 +02:00
combo/apps/wcs/static/js/combo.wcs-trigger-button.js Outdated
@ -0,0 +1,60 @@
class WcsTriggerButton extends HTMLElement {
#triggerUrl
fpeters commented 2024-04-26 09:43:25 +02:00
Owner
Copy Link

Désolé question javascript, ça correspond à quoi ce #triggerUrl ?

Désolé question javascript, ça correspond à quoi ce #triggerUrl ?
csechet commented 2024-04-26 09:56:26 +02:00
Author
Owner
Copy Link

C'est des variables privées en JS (vraiment privée, ça lève une erreur de syntaxe si on y accède en dehors de la classe, ou si elle n'est pas déclarée dans la classe).

Pas vraiment d'avis définitif sur si c'est bien ou pas, c'est probablement une habitude que je tiens du C++ : je ne vois pas d'inconvénient à en faire des membres "standard".

C'est des variables privées en JS (vraiment privée, ça lève une erreur de syntaxe si on y accède en dehors de la classe, ou si elle n'est pas déclarée dans la classe). Pas vraiment d'avis définitif sur si c'est bien ou pas, c'est probablement une habitude que je tiens du C++ : je ne vois pas d'inconvénient à en faire des membres "standard".
combo/apps/wcs/static/js/combo.wcs-trigger-button.js Outdated
@ -0,0 +6,4 @@
const label = this.getAttribute('label')
this.#triggerUrl = this.getAttribute('trigger-url')
this.innerHTML = `<button>${label}</button>`
if(this.#triggerUrl === null) {
fpeters commented 2024-04-26 09:43:59 +02:00
Owner
Copy Link

Et pareil question js, .# c'est quoi ?

(et détail je trouve plus lisible quand il y a un espace après le if).

Et pareil question js, .# c'est quoi ? (et détail je trouve plus lisible quand il y a un espace après le if).
csechet commented 2024-04-26 09:57:08 +02:00
Author
Owner
Copy Link

Et pareil question js, .# c'est quoi ?

C.F commentaire précédent.

(et détail je trouve plus lisible quand il y a un espace après le if).

Je vais peut-être en profiter pour faire tourner eslint sur les nouveaux fichiers ajoutés.

> Et pareil question js, .# c'est quoi ? C.F commentaire précédent. > (et détail je trouve plus lisible quand il y a un espace après le if). Je vais peut-être en profiter pour faire tourner eslint sur les nouveaux fichiers ajoutés.
combo/apps/wcs/static/js/combo.wcs-trigger-button.js Outdated
@ -0,0 +47,4 @@
const button = this.querySelector('button')
if(!response.ok || json.err === 1) {
alert('An error occured, please retry later.')
fpeters commented 2024-04-26 09:42:53 +02:00
Owner
Copy Link

Passer ça dans gettext ?

Passer ça dans gettext ?
combo/apps/wcs/templates/combo/wcs/manager/card-cell-form-display-as-card.html Outdated
@ -189,0 +191,4 @@
<div data-dynamic-display-child-of="entry_type" data-dynamic-display-value="@action@">
<p>
<label>
{% trans "Label" %}
fpeters commented 2024-04-26 09:45:44 +02:00
Owner
Copy Link

: manquant ?

: manquant ?
combo/apps/wcs/templates/combo/wcs/manager/card-cell-form-display-as-card.html Outdated
@ -189,0 +197,4 @@
</p>
<p>
<label>
{% trans "Action :" %}
fpeters commented 2024-04-26 09:45:56 +02:00
Owner
Copy Link

Espace à retirer avant le :

Espace à retirer avant le :
combo/apps/wcs/templates/combo/wcs/manager/card-cell-form-display-as-card.html Outdated
@ -189,0 +207,4 @@
</p>
<p>
<label>
{% trans "Ask confirmation " %}
fpeters commented 2024-04-26 09:46:22 +02:00
Owner
Copy Link

Espace à remplacer par un :

Espace à remplacer par un :
combo/apps/wcs/templates/combo/wcs/manager/card-cell-form-display-as-card.html Outdated
@ -189,0 +213,4 @@
</p>
<p data-dynamic-display-child-of="action_ask_confirmation" data-dynamic-display-value-in=" True ">
<label>
{% trans "Confirmation text (Template)" %}
fpeters commented 2024-04-26 09:46:43 +02:00
Owner
Copy Link

Je retirerais la majuscule à Template, et j'ajouterais un :

Je retirerais la majuscule à Template, et j'ajouterais un :
combo/apps/wcs/templates/combo/wcs/manager/card-cell-form-display-as-card.html Outdated
@ -189,0 +219,4 @@
</p>
<p>
<label>
{% trans "Unavailable action mode :" %}
fpeters commented 2024-04-26 09:46:57 +02:00
Owner
Copy Link

Espace à retirer devant les :

Espace à retirer devant les :
combo/apps/wcs/templates/combo/wcs/manager/card-cell-form-display-as-table.html Outdated
@ -154,0 +163,4 @@
</p>
<p>
<label>
{% trans "Label" %}
fpeters commented 2024-04-26 09:48:06 +02:00
Owner
Copy Link

cf commentaires plus haut.

J'imagine que c'était galère de factoriser pour ne pas dupliquer ça, qu'il y a des petites différences ?

cf commentaires plus haut. J'imagine que c'était galère de factoriser pour ne pas dupliquer ça, qu'il y a des petites différences ?
csechet commented 2024-04-26 09:58:26 +02:00
Author
Owner
Copy Link

J'ai jeté un œil pour factoriser, les templates et le code dans combo.manager.js, mais vu l'existant c'est une grosse galère : j'ai préféré coller à ce qui était déjà fait, même si ça me satisfait pas vraiment non plus.

J'ai jeté un œil pour factoriser, les templates et le code dans combo.manager.js, mais vu l'existant c'est une grosse galère : j'ai préféré coller à ce qui était déjà fait, même si ça me satisfait pas vraiment non plus.
combo/apps/wcs/urls.py Outdated
@ -26,4 +26,5 @@ urlpatterns = [
redirect_crypto_url,
name='wcs-redirect-crypto-url',
),
re_path(r'^api/wcs/trigger/$', TriggerProxyView.as_view(), name='wcs-trigger-proxy'),
fpeters commented 2024-04-26 09:48:47 +02:00
Owner
Copy Link

Ça peut se faire avec path() tout simplement, plutôt que re_path().

Ça peut se faire avec path() tout simplement, plutôt que re_path().
combo/manager/static/js/combo.manager.js Outdated
@ -435,2 +432,2 @@
$(sel1 + sel2).removeClass('field-hidden').show();
$(sel1 + sel3).removeClass('field-hidden').show();
if(this.getAttribute('type') == 'checkbox') {
fpeters commented 2024-04-26 09:51:38 +02:00
Owner
Copy Link

Dans wcs le cas des cases à cocher est déjà géré, différemment, (cf wcs/qommon/static/js/qommon.js), ça serait bien de poser le même fonctionnement ici, pour faciliter à un moment la convergence dans gadjo.

Dans wcs le cas des cases à cocher est déjà géré, différemment, (cf wcs/qommon/static/js/qommon.js), ça serait bien de poser le même fonctionnement ici, pour faciliter à un moment la convergence dans gadjo.
csechet force-pushed wip/75908-wcs-trigger from 03a6bfc328 to 981778af4d 2024-04-29 14:00:09 +02:00 Compare
csechet force-pushed wip/75908-wcs-trigger from 981778af4d to 120ea4a802 2024-05-03 09:36:02 +02:00 Compare
All checks were successful
gitea/combo/pipeline/head This commit looks good
Details
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
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/combo#252
No description provided.
Delete Branch "wip/75908-wcs-trigger"

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?