cisstech / platon

Platform for Learning and Teaching Online
Other
14 stars 3 forks source link

Component documentation Type Implementation #16

Closed Ofghanirre closed 1 year ago

Ofghanirre commented 1 year ago

Hello ! J'ai vu qu'on pouvait définir dans les documentations des Webcomposants les différentes valeurs possibles, sur un champs à l'aide d'un champs enum, je propose un quickFix permettant d'afficher les valeurs possibles à la place du type dans la documentation si il est bien définit.

Permettant de passer de cet affichage:

Pris de la documentation du Widget Feedback : https://platon-preprod.univ-eiffel.fr/docs/pages/widgets/wc-feedback image

vers celui ci: image

C'est inspiré de la docs présente sur des libs comme ng-zorro, et je trouve ça assez pratique, qu'en penses tu ?

mciissee commented 1 year ago

Merci pour la PR, je te merge le code après les vérifications du CI. Au passage du as un problème avec la configuration de ton ESLINT qui n'est pas le même que celui du projet

codecov-commenter commented 1 year ago

Codecov Report

Merging #16 (4f0aadc) into main (0ba89be) will not change coverage. The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@           Coverage Diff           @@
##             main      #16   +/-   ##
=======================================
  Coverage   89.13%   89.13%           
=======================================
  Files           2        2           
  Lines          46       46           
  Branches       11       11           
=======================================
  Hits           41       41           
  Misses          1        1           
  Partials        4        4           

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more