Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add an EJS Format #2047

Merged
merged 11 commits into from
Jun 21, 2024
Merged

Add an EJS Format #2047

merged 11 commits into from
Jun 21, 2024

Conversation

AlasDiablo
Copy link
Collaborator

No description provided.

@AlasDiablo AlasDiablo added the 👷‍♂️ Ready For Review PR en attente de relecture et de validation label Jun 18, 2024
Copy link
Contributor

@touv touv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merci, c’est bon début, il reste quelques détails pour que ce format puisse être compris facilement par des non experts, j’ajouterais :

  • les traductions pour le nom et la description du format
  • un texte explicatif au dessus de l’éditeur, pour expliquer ce qui attendu, cad un template HTML respectant la syntaxe EJS
  • La zone de saisie ne devrait pas être vide par défaut, j’y mettrai un exemple qui fonctionne avec la majorité des routines. Exemple :
<%# Template EJS de mis en forme des données retournées par une routine %>
<table>
    <thead>
    <tr>
      <th>ID</th>
      <th>Value</th>
    </tr>
  </thead>
  <tbody>
    <% formatData.forEach((value) => { %>
    <tr>
        <td><%= value._id %></td>
        <td><%= value.value %></td>
    </tr>
  <% }); %>    
  </tbody>
</table>

  • je n’utiliserai formatData pas comme nom de variable principale, mais root.values comme dans le format JSONDebug, j'ajouterai une référence à ce format pour inviter à découvrir et visualiser les données qui devront être mise en forme par le template

  • en plus de formatData, j’atouterais formatTotal (root.total) et formatError (root.Error), en les renommant de manière cohérente par rapport à formatData

  • En cas d’erreur, j’afficherai un message dans la console du navigateur

@AlasDiablo AlasDiablo self-assigned this Jun 19, 2024
@AlasDiablo AlasDiablo requested review from parmentf and touv June 19, 2024 09:30
src/app/custom/translations.tsv Outdated Show resolved Hide resolved
src/app/custom/translations.tsv Outdated Show resolved Hide resolved
src/app/custom/translations.tsv Outdated Show resolved Hide resolved
src/app/js/formats/text/ejs/EJSAdmin.js Outdated Show resolved Hide resolved
src/app/js/formats/text/ejs/example.ejs Outdated Show resolved Hide resolved
src/app/js/formats/text/ejs/example.ejs Outdated Show resolved Hide resolved
src/app/js/formats/text/ejs/example.ejs Outdated Show resolved Hide resolved
src/app/js/formats/text/ejs/example.ejs Outdated Show resolved Hide resolved
src/app/js/formats/text/ejs/example.ejs Outdated Show resolved Hide resolved
src/app/js/formats/text/ejs/example.ejs Outdated Show resolved Hide resolved
AlasDiablo and others added 2 commits June 20, 2024 07:49
@AlasDiablo AlasDiablo requested a review from parmentf June 20, 2024 05:51
@touv
Copy link
Contributor

touv commented Jun 20, 2024

L'exemple par défaut fonctionne parfaitement avec tous les formats produits par les routines. C'est un avantage.
Mais du coup, il est complexe on, s’éloigne d'un exemple simple et facile à comprendre.

Ne serait-il pas possible d'avoir des exemples séparés ?
On afficherait un exemple simple prévu pour fonctionner avec { _id, value } et les autres seraient chargés en cliquant sur un lien explicite.

src/app/custom/translations.tsv Outdated Show resolved Hide resolved
Co-authored-by: François Parmentier <francois.parmentier@gmail.com>
@touv touv merged commit b6592c9 into master Jun 21, 2024
9 checks passed
@touv touv deleted the feat/format/ejs branch June 21, 2024 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
👷‍♂️ Ready For Review PR en attente de relecture et de validation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants