Skip to content
Snippets Groups Projects

Draft: #33

Merged SMAH Lisa requested to merge 33-implementer-la-navigation-apres-la-recherche-plein-texte into dev

#34 (closed) Ajout d'une navigation avec des onglets "Rechercher", "Explorer", "À propos" et "Mentions légales"

Closes #33 (closed)

Merge request reports

Merged by Juliette Engelaere-LefebvreJuliette Engelaere-Lefebvre 9 months ago (Nov 6, 2024 3:38pm UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • 381 "Aucun lot de données ne correspond à votre recherche, veuillez renouveler votre recherche ou explorer les bases de données disponibles (2e onglet)",
    340 382 )
    341 } else {
    342 NULL
    343 }
    383 )
    384 } else {
    385 NULL
    386 }
    387 })
    388 })
    389
    390
    391
    392 # Réagir à la sélection d'une base
    393 observeEvent(input$selected_base, {
    • ici tu recodes tout ce qui se passe à partir de l'onglet explorer, ce n'est pas maintenable dans le temps. il faut parvenir à ce que les 2 inputs input$selected_base et input$base déclenchent les mêmes opérations Ce passage sera à revoir (pour le supprimer) une fois la réactivité mise au carré. il y a un bloc à introduire du type

      observeEvent(input$selected_base) {
      r$base <- input$selected_base
      }
      
      observeEvent(input$base) {
      r$base <- input$base
      }

      et ensuite utiliser le r$base dans les opérations suivantes d'affichage des informations

    • Please register or sign in to reply
  • Lisa, peux-tu revoir la réactivité pour adopter les bonnes pratiques explicitées dans mon mail et également transformer les listes de choix bases/schémas en vecteur nommé ? Je remets la demande de MR en brouillon, n'hésite pas à demander de l'aide au besoin.

    Edited by Juliette Engelaere-Lefebvre
  • requested changes

  • Juliette Engelaere-Lefebvre marked this merge request as draft

    marked this merge request as draft

  • SMAH Lisa added 1 commit

    added 1 commit

    Compare with previous version

  • SMAH Lisa added 1 commit

    added 1 commit

    Compare with previous version

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading