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

[BUGFIX] Ordonner les skillName afin de garantir l'ordre des colonnes lors de l'export de résultat (Pix-16196) #11220

Merged
merged 1 commit into from
Jan 29, 2025

Conversation

xav-car
Copy link
Contributor

@xav-car xav-car commented Jan 24, 2025

🥞 Problème

certains export comporte les nom des acquis pour des traitement plus "fin" . le souci c'est que nous ne garantissons pas l'ordre entre différent export

🥓 Proposition

Trier ces noms d'acquis par rapport à sa compétence, puis par son nom.

🧃 Remarques

RAS

😋 Pour tester

CI au vert,

Allez sur PixAdmin :

  • activé l'affichage des acquis dans l'export de résultat.

Allez sur PixOrga :

  • Faire un export CSV d'une campagne d'évaluation et vérifier que l'ordre des acquis sont là

@xav-car xav-car added 👀 Tech Review Needed 👀 Func Review Needed Need PO validation for this functionally team-prescription ⚠️ PR Inheritance This PR inherits a first-to-merge PR and will need a rebase labels Jan 24, 2025
@xav-car xav-car self-assigned this Jan 24, 2025
@xav-car xav-car requested a review from a team as a code owner January 24, 2025 15:03
@xav-car xav-car changed the base branch from dev to pix-16195/campaign-learning-content-refacto January 24, 2025 15:04
@alicegoarnisson
Copy link
Contributor

Question : sur cette PR, dans les profils cibles, on a des sujets triés par thématiques, et dans le fichier d'export, les sujets n'apparaissent pas dans le même ordre, ils sont triés par ordre alphabétique à l'intérieur de la compétence.
Comment on faisait avant ? Avait-on un ordre défini pour les sujets dans le fichier d'export ?

@alicegoarnisson
Copy link
Contributor

alicegoarnisson commented Jan 27, 2025

Func OK, tech OK 🦦 🐼

@@ -15,6 +15,10 @@ class CampaignLearningContent extends LearningContent {
return super.competences.sort((a, b) => a.index.localeCompare(b.index));
}

get skills() {
return this.competences.flatMap((competence) => competence.skills.sort((a, b) => a.name.localeCompare(b.name)));
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Question: pourquoi ne pas faire un super.skills ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pour avoir les skills trié par compétence. le super.skills ne se base pas sur les compétence pour le trier. Du coup on aurait potentiellement des skills de la competence 4 avant la compétence 1 etc....

@alicegoarnisson alicegoarnisson added Func Review OK PO validated functionally the PR Tech Review OK and removed 👀 Tech Review Needed 👀 Func Review Needed Need PO validation for this functionally labels Jan 27, 2025
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-16195/campaign-learning-content-refacto branch from fc20a35 to 5fa42d0 Compare January 29, 2025 09:18
Base automatically changed from pix-16195/campaign-learning-content-refacto to dev January 29, 2025 09:24
@xav-car xav-car removed the ⚠️ PR Inheritance This PR inherits a first-to-merge PR and will need a rebase label Jan 29, 2025
@xav-car xav-car force-pushed the pix-16196/order-skill-export-result branch from dbfe8b6 to c636060 Compare January 29, 2025 09:49
@xav-car
Copy link
Contributor Author

xav-car commented Jan 29, 2025

Question : sur cette PR, dans les profils cibles, on a des sujets triés par thématiques, et dans le fichier d'export, les sujets n'apparaissent pas dans le même ordre, ils sont triés par ordre alphabétique à l'intérieur de la compétence. Comment on faisait avant ? Avait-on un ordre défini pour les sujets dans le fichier d'export ?

Analyse trie par "réussite" du coup il semblerait que nous ne soyons pas trop impacté sur cette page là. par contre je ne vois pas ce que tu entend pas "dans les profil cible" ?

@alicegoarnisson
Copy link
Contributor

Question : sur cette PR, dans les profils cibles, on a des sujets triés par thématiques, et dans le fichier d'export, les sujets n'apparaissent pas dans le même ordre, ils sont triés par ordre alphabétique à l'intérieur de la compétence. Comment on faisait avant ? Avait-on un ordre défini pour les sujets dans le fichier d'export ?

Analyse trie par "réussite" du coup il semblerait que nous ne soyons pas trop impacté sur cette page là. par contre je ne vois pas ce que tu entend pas "dans les profil cible" ?

En fait je suis allée voir le PC d'une campagne sur Admin (la SCO SIECLE, de mémoire), et je vois qu'il y a des sujets qui apparaissent dans un ordre donné. Comme l'ordre dans le fichier d'export n'était pas le même, je me suis demandée si c'était normal.
Mais si on tri par réussite alors c'est bueno

@xav-car
Copy link
Contributor Author

xav-car commented Jan 29, 2025

Question : sur cette PR, dans les profils cibles, on a des sujets triés par thématiques, et dans le fichier d'export, les sujets n'apparaissent pas dans le même ordre, ils sont triés par ordre alphabétique à l'intérieur de la compétence. Comment on faisait avant ? Avait-on un ordre défini pour les sujets dans le fichier d'export ?

Analyse trie par "réussite" du coup il semblerait que nous ne soyons pas trop impacté sur cette page là. par contre je ne vois pas ce que tu entend pas "dans les profil cible" ?

En fait je suis allée voir le PC d'une campagne sur Admin (la SCO SIECLE, de mémoire), et je vois qu'il y a des sujets qui apparaissent dans un ordre donné. Comme l'ordre dans le fichier d'export n'était pas le même, je me suis demandée si c'était normal. Mais si on tri par réussite alors c'est bueno

Effectivement, ce n'est pas trié côté PixAdmin. peut être que ça vaudrait le coup de le faire dans un second temps pour que nos usagers interne s'y retrouve. après on pourrait se dire que c'est déjà trié par compétence / tubes . pas besoin d'aller plus loin sur PixAdmin

@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-16196/order-skill-export-result branch from c636060 to 358d54e Compare January 29, 2025 13:40
@pix-service-auto-merge pix-service-auto-merge merged commit 350a2de into dev Jan 29, 2025
8 of 10 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-16196/order-skill-export-result branch January 29, 2025 13:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants