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

130 composant implementation de fileinput #133

Merged
merged 9 commits into from
Jul 26, 2023

Conversation

jengelaere
Copy link
Contributor

@jengelaere jengelaere commented Jul 10, 2023

feat : ajout fileInput_dsfr et tableOutput_dsfr

  • Est-ce que mon code respecte les standards de qualité de mise en production de packages ?
  • Est-ce que la personne qui révise à toutes les informations pour valider les fonctionnalités / résolutions de problèmes sans trop de recherches ?
  • Est-ce que le client qui validera les tickets associés à les informations pour le faire sans perte de temps ?

Issues à faire valider pour fermer :

Issues traitées à garder ouvertes ou en cours :

  • issue #

Checklist:

  • Est-ce que le check du package passe en local ? 0 errors ✔ | 0 warnings ✔ | 4 notes ✖
  • Est-ce que le CI passe ?
  • Est-ce que les fonctionnalités ajoutées / corrigées, sont documentées, testées ?
  • Est-ce que les fonctionnalités ajoutées / problèmes résolus sont brièvement présentées dans le message de la MR ?
  • Est-ce que les modifications sont liées à des tickets / issues que j'ai listés dans les commits et dans la MR elle-même ?
  • Est-ce que les tickets sont en mode "révision" dans le Board de suivi du projet ?
  • Est-ce que chaque ticket, s'il doit être fermé après acceptation de la MR contient un commentaire qui dit comment le valider ?

Implémentation de fileInput_dsfr_template(), fileInput_dsfr(), tableOutput() :
- fonctions documentées avec exemples
- tests
- implementation dans l'app de demo, partie input d'actions
- table de passage pour convert_to_dsfr

ce faisant, reprise exmeple selectInput_dsfr sur les arguments de header_dsfr qui ont bougés

ticket #130
fix typo DESCRIPTION
@jengelaere jengelaere linked an issue Jul 10, 2023 that may be closed by this pull request
4 tasks
jengelaere and others added 6 commits July 26, 2023 09:10
pour tenter de réparer le déploiement continu
…nt déployés vers shinyApps et qui ressortaient en alertes de contenu
…sconnect.packrat = TRUE) pour restaurer son comportement précédent depuis le passage à la v1.0.0
@jengelaere jengelaere marked this pull request as draft July 26, 2023 07:22
@jengelaere jengelaere marked this pull request as ready for review July 26, 2023 07:23
Merge branch 'main' into 130-composant-implementation-de-fileinput

# Conflicts:
#	NAMESPACE
#	inst/v1.9.3/table_correspondance_shiny_dsfr.csv
@codecov-commenter
Copy link

Codecov Report

Merging #133 (1cd4813) into main (330c7cb) will increase coverage by 0.27%.
The diff coverage is 88.09%.

❗ Current head 1cd4813 differs from pull request most recent head 3c8dad5. Consider uploading reports for the commit 3c8dad5 to get more accurate results

@@            Coverage Diff             @@
##             main     #133      +/-   ##
==========================================
+ Coverage   78.51%   78.78%   +0.27%     
==========================================
  Files          60       63       +3     
  Lines        1438     1480      +42     
==========================================
+ Hits         1129     1166      +37     
- Misses        309      314       +5     
Files Changed Coverage Δ
R/actionbutton_dsfr.R 100.00% <ø> (ø)
R/mod_input_actions.R 78.65% <78.26%> (-0.14%) ⬇️
R/fileinput_dsfr.R 100.00% <100.00%> (ø)
R/fileinput_dsfr_template.R 100.00% <100.00%> (ø)
R/tableoutput_dsfr.R 100.00% <100.00%> (ø)

@jengelaere jengelaere merged commit 310678e into main Jul 26, 2023
13 checks passed
@jengelaere jengelaere deleted the 130-composant-implementation-de-fileinput branch July 26, 2023 10:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[composant] Implémentation de fileInput
4 participants