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

Mes dossiers #147

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Mes dossiers #147

wants to merge 6 commits into from

Conversation

DavidBruant
Copy link
Collaborator

No description provided.

@DavidBruant DavidBruant requested a review from Ynote January 6, 2025 10:50
/** @type {Set<NonNullable<Personne['email']> | AUCUN_INSTRUCTEUR>}*/
const instructeursOptions = new Set([AUCUN_INSTRUCTEUR, ...instructeurEmailOptions])
/** @type {Set<NonNullable<Personne['email']> | AUCUN_INSTRUCTEUR>} */
const instructeursOptions = new Set([email, AUCUN_INSTRUCTEUR, ...instructeurEmailOptions])
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

On rajoute l'email de l'instructeur actuel en premier


/** @type {Set<NonNullable<Personne['email']> | AUCUN_INSTRUCTEUR>} */
$: instructeursSélectionnés = new Set()
let instructeursSélectionnés = new Set([email])
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

N'afficher que les dossiers de l'instructeur actuel par défaut

mettreÀJourOptionsSélectionnées={filtrerParInstructeurs}
/>
{/if}
{#if dossiersIdSuivisParInstructeurActuel && dossiersIdSuivisParInstructeurActuel.size >= 1}
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Suppression du bouton "Afficher uniquement mes dossiers suivis" qui n'a plus d'utilité

@@ -45,7 +44,7 @@
})()

/** @type {(DossierComplet & {évènementsPhase: ÉvènementPhaseDossier[]})[]} */
$: dossiersSelectionnés = dossiers
$: dossiersSelectionnés = []
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

amélioration de la perf
si on met tous les dossiers, il y a un premier rendu qui coûte cher en perf
SI on met [], c'est beaucoup plus rapide

Copy link
Collaborator

@Ynote Ynote left a comment

Choose a reason for hiding this comment

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

Il y a juste un mini-import qui n'a pas d'utilité. J'ai mis une suggestion pour le retirer.

C'est très cool de n'avoir que ses propres dossiers au démarrage, alors que je ne fais pas d'instruction et qu'on a que deux dossiers dans le groupe dev. J'imagine que ça va être bien pour les instructrices aussi.

Copy link

Svelte Check Results

No issues found! 🎉


Last Updated: 17th January at 15:47 (858b609)

@DavidBruant
Copy link
Collaborator Author

C'est très cool de n'avoir que ses propres dossiers au démarrage, alors que je ne fais pas d'instruction et qu'on a que deux dossiers dans le groupe dev. J'imagine que ça va être bien pour les instructrices aussi.

Oui, c'est clairement une meilleure idée comme comportement par défaut

Quand j'ai montré pitchou à une instructrice, c'était sa première demande (ne voir que ses dossiers)

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