entrouvert
/
publik-base-theme
15
1
Fork 1
Code Issues Pull Requests 11 Releases Wiki Activity

WIP: dpark: permettre de paramétrer le titre et de donner un libellé au compte Publik (71302) #17

Draft
nroche wants to merge 2 commits from wip/71302-toulouse-2022-dpark into main
pull from: wip/71302-toulouse-2022-dpark
merge into: entrouvert:main
Conversation 2 Commits 2 Files Changed 3 +17 -3
nroche commented 2022-11-15 17:52:50 +01:00
Owner
Copy Link

0001: avoir le titre de paramétrable.

0002: ajout d'un gabarit pour modifier le texte.
(ça m'a semblé plus simple que de multiplier les variables ou d'ajouter ce texte dans le connecteur)

0001: avoir le titre de paramétrable. 0002: ajout d'un gabarit pour modifier le texte. (ça m'a semblé plus simple que de multiplier les variables ou d'ajouter ce texte dans le connecteur)
nroche force-pushed wip/71302-toulouse-2022-dpark from 29dc66b207 to 0ac23ed265 2022-11-15 18:01:21 +01:00 Compare
fpeters requested changes 2022-11-15 18:08:42 +01:00
templates/combo/json/dpark.html Outdated
@ -2,3 +2,4 @@
<div>
{% if not json.data %}
{% block empty_data_text %}
<div><p>Vous pouvez relier votre compte Publik à votre compte D-Park via <a href="{{pairing_form_url}}">ce formulaire</a>.</p></div>
fpeters commented 2022-11-15 18:02:07 +01:00
Owner
Copy Link

Perso il aurait juste fallu de base supprimer Publik (et D-Park) ici; en l'état ce template générique évidemment parle peu aux gens; reste que maintenant qu'il y a eu une demande pour quelque chose de plus particulier, il va falloir le faire, mais

Perso il aurait juste fallu de base supprimer Publik (et D-Park) ici; en l'état ce template générique évidemment parle peu aux gens; reste que maintenant qu'il y a eu une demande pour quelque chose de plus particulier, il va falloir le faire, mais
templates/variants/toulouse-2022/combo/json/dpark.html Outdated
@ -0,0 +1,5 @@
{% extends "combo/json/dpark.html" %}
{% block empty_data_text %}
<div><p>Vous pouvez relier votre compte MonToulouse à votre abonnement stationnement via <a href="{{pairing_form_url}}">ce formulaire</a>.</p></div>
fpeters commented 2022-11-15 18:08:17 +01:00
Owner
Copy Link

peut-être que quand même "abonnement statitionnement" pourrait être monté dans le gabarit générique; et ne resterait en spécifique qu'un (nouveau) bloc qui contiendrait juste "MonToulouse" ?

peut-être que quand même "abonnement statitionnement" pourrait être monté dans le gabarit générique; et ne resterait en spécifique qu'un (nouveau) bloc qui contiendrait juste "MonToulouse" ?
nroche commented 2022-11-15 18:29:16 +01:00
Author
Owner
Copy Link

ne resterait en spécifique qu'un (nouveau) bloc qui contiendrait juste "MonToulouse" ?

Autant mettre le libellé du compte (ici MonToulouse) dans un nouveau champ de formulaire alors (comme fait sur le premier patch) ?
(pour ne pas ajouter de gabarit pour toulouse ni le bloc)

> ne resterait en spécifique qu'un (nouveau) bloc qui contiendrait juste "MonToulouse" ? Autant mettre le libellé du compte (ici MonToulouse) dans un nouveau champ de formulaire alors (comme fait sur le premier patch) ? (pour ne pas ajouter de gabarit pour toulouse ni le bloc)
nroche force-pushed wip/71302-toulouse-2022-dpark from 0ac23ed265 to e0475e307b 2022-11-16 09:02:37 +01:00 Compare
nroche commented 2022-11-16 09:06:09 +01:00
Author
Owner
Copy Link

Dans le doute j'ai appliqué les changements proposés par Fred.

Dans le doute j'ai appliqué les changements proposés par Fred.
nroche changed title from wip/71302-toulouse-2022-dpark to WIP: dpark: permettre de paramétrer le titre et de donner un libellé au compte Publik (71302) 2023-02-23 11:54:23 +01:00
All checks were successful
gitea-wip/publik-base-theme/pipeline/pr-main This commit looks good
Details
gitea/publik-base-theme/pipeline/pr-main This commit looks good
Details
gitea/publik-base-theme/pipeline/head This commit looks good
Details
This pull request is marked as a work in progress.
This branch is out-of-date with the base branch
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b wip/71302-toulouse-2022-dpark main
git pull origin wip/71302-toulouse-2022-dpark

Step 2:

Merge the changes and update on Gitea.
git checkout main
git merge --no-ff wip/71302-toulouse-2022-dpark
git push origin main
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/publik-base-theme#17
No description provided.
Delete Branch "wip/71302-toulouse-2022-dpark"

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?