-
Notifications
You must be signed in to change notification settings - Fork 4
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
feat: ajout filtre secteur professionnel #3155
Conversation
import { read, utils } from "xlsx"; | ||
|
||
async function main() { | ||
const res = await axios.get("https://www.pole-emploi.fr/files/live/sites/PE/files/ROME_ArboPrincipale.xlsx", { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bonus : éventuellement en constante si jamais ca bouge
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
En soi, ce literal est une constante 😁
Comme cette URL n'est utilisée qu'à un seul endroit et qu'elle ne risque pas de changer prochainement, j'ai préféré ne pas mettre de variable intermédiaire.
Bien sûr, si un jour on doit être amenés à changer cette URL manuellement et régulièrement, alors on pourra retoucher cette partie pour mettre plus en évidence les parties variables qui doivent être configurées.
Sachant que si l'URL change, il y aura probablement un changement de format, et dans ce cas le script en entier sera alors à revoir.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok pour moi
? // hack: reverse pour faire en sorte que l'id root soit en dernier sinon la lib sélectionne | ||
// le dernier élément et on perd le focus de la recherche |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍🏼
🎉 This PR is included in version 3.30.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Contenu :
Bug :
Au déploiement, il faudra lancer les script :
J'ai ajouté des dépendances côté serveur liés au parsing xml + décompression zip. Idéalement, ça devrait sortir du serveur mais comme on est très monolithique... voilà. 🙃