COM-3475: can access course as holding admin

[ulysseferreira]

TESTS 💻

• J'ai codé les tests unitaires -np
• J'ai codé les tests d'intégration
• C'est une ancienne route utilisée par les apps mobiles.
  • Si oui, J'ai fait de nouveaux tests sans modifier les anciens

• Je replie cette section car mes modifications n'ont pas besoin de tests unitaires ou d'intégration

---

POINTS D'ATTENTION POUR CETTE PR ⚠️

• J'ai fait des modifications sur une route utilisée sur plusieurs plateformes Doc de détail
• J'ai modifié un modèle utilisé en mobile Doc de détail
• J'ai ajouté un modèle spécifique à une structure
  • Si oui, j'ai ajouté le champs company ainsi que les prehooks associés
• J'ai ajouté/modifié une constante qui est utilisée sur les apps mobile
  • Si oui, j'ai précisé sur le Doc de MEP qu'il faut forcer la mise à jour
• J'ai ajouté une variable d'environnement
  • Si oui, J'ai précisé sur le Doc de MES et Doc de MEP les modifications faites

• Je replie cette section car je n'ai pas fait de modifications entrainant un point d'attention

---

FONCTIONNALITÉS APPS MOBILES 📱

• Mes changements impactent l'application formation
  • Si oui, j'ai testé que les anciennes versions maintenues fonctionnent toujours
• Mes changements impactent l'application erp
  • Si oui, j'ai testé que les anciennes versions maintenues fonctionnent toujours

• Je replie cette section car mes modifications n'ont pas d'impact sur les applications mobiles

Si tu as lu cette description, pense à réagir avec un 👁️

[sophiemoustard]

j'ai l'impression que ca fait quelque fois qu'on a besoin de regarder s'il y a une intersection entre 2 tableaux et ca risque de revenir souvent avec ce chantier. que penses tu de créer une méthode doIdArraysIntersect / doArraysOfIdsIntersect (ou un autre nom si tu as plus d'inspiration ^^) ?

[sophiemoustard]

en faisant un recherche avec cette regex dans vsc \.some\([a-zA-Z]+ => UtilsHelper\.doesArrayIncludeId, il y a 5 endroits qujourd'hui ou on pourrait l'utiliser (ici + 4 fois dans seedVerification)

[ulysseferreira]

done