Skip to content

Conversation

@ddotta
Copy link
Contributor

@ddotta ddotta commented Apr 4, 2022

Bon je tente une PR pour ajouter les liens vers les pages d'Insee.fr dans donnees_dispo().
cf. cette discussion #71
Je ne suis pas sûr d'avoir adopté la meilleure stratégie dans les modifications du package mais ça marche au final donc je vous laisse voir si ça vous demande beaucoup de corrections ou pas

Les principales modifications apportées :

  • Ajout d'une colonne dans la data.frame en sortie de donnees_dispo(). Son contenu utilise consulter()
  • Ajout d'un argument consultation (booléen) dans consulter() (pour éviter d'avoir le navigateur qui s'affole pour chaque appel à consulter()
  • même chose dans infoDonnees() pour éviter d'avoir le message "Sélection automatique des données les plus récentes" qui s'affiche systématiquement
  • Ajout de params$escape <- FALSE dans les paramètres du datatable pour un bon affichage des URLs

@py-b
Copy link
Member

py-b commented Apr 9, 2022

@ddotta : merci, c'est nickel. J'ai juste amélioré quelques petites choses.

@py-b py-b merged commit 8153ac7 into InseeFrLab:master Apr 9, 2022
@ddotta ddotta deleted the lien-page branch April 11, 2022 08:24
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.

2 participants