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

Ajout d'un filtre activité principale sur le tableau de suivi #135

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

Conversation

…activite

* 'main' of github.com:betagouv/pitchou:
  Mise à jour schema DS 88444 + refacto pitchouKeyToAnnotationDS + correction bug département quand commune vide (#136)
…activite

* 'main' of github.com:betagouv/pitchou:
  Permettre le tri par ordre alphabétique sur certaines colonnes du tableau de suivi (#127)
  Tweaks outil sync (#134)
  Couleurs et premier jet procédure (#128)
…activite

* 'main' of github.com:betagouv/pitchou:
  Suppression ts-to-jsdoc (#142)
  Renomme phase verif ddep (#143)
  Dédoublonnage des espèces impactées dans l'aide à la rédaction d'arrêtés (#144)
  Réduction de la largeur tableau suivi (#139)
  Supprimer action attendue (#138)
  Stocker le département principal si ce dernier est rempli en cas d'absence des autres infos de localisation (#137)
  Update description-pitchou.md
  Documentation jekyll pages (#141)
@Ynote Ynote marked this pull request as ready for review January 2, 2025 16:10
@@ -134,10 +138,12 @@
* @param {string} _texteÀChercher
*/
function filtrerParTexte(_texteÀChercher) {
const texteÀChercherSansEspace = _texteÀChercher.trim()
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@Ynote Ynote requested a review from DavidBruant January 2, 2025 16:12
Copy link
Collaborator

@DavidBruant DavidBruant left a comment

Choose a reason for hiding this comment

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

je ne comprends pas bien la danse avec le téléchargement de fichier, la génération d'un json, etc.
Le schema est déjà dans le store côté front-end (store.state.schemaDS88444)
et donc, on a déjà la liste des activités prinicipales côté client

(je réalise que c'est ptèt un peu absurde de télécharger le schéma systématiquement pour toutes les pages alors que jusque là, il ne servait que pour les liens de pré-remplissage, mais 🤷, il est là, donc on peut l'utiliser sans générer un fichier intermédiaire )

Je veux bien en repapoter avant de faire une review plus détaillée

Ynote added 2 commits January 21, 2025 12:38
…activite

* 'main' of github.com:betagouv/pitchou:
  ajout de svelte-check dans le CI (#149)
  Normalisation nom base de données pour simplifier les restore (#148)
Copy link

github-actions bot commented Jan 21, 2025

Svelte Check Results

Found 1 error and 0 warnings (1 total) with the files in this PR.

scripts/front-end/routes/Accueil.js

scripts/front-end/routes/Accueil.js:95:12

Error: Type '{ dossiers: (DossierComplet & { évènementsPhase: VNementPhaseDossier[]; })[]; relationSuivis: Map<string, Set<DossierId>> | undefined; activitésPrincipales: DossierDemarcheSimplifiee88444["Activit\u00E9 principale"][] | undefined; nav?: boolean | undefined; email?: string | undefined; erreurs?: Set<...> | undefined; }' is not assignable to type '{ email: string; relationSuivis: Map<string, Set<DossierId>> | undefined; erreurs: Set<{ message: string; }> | undefined; dossiers?: (DossierComplet & { ...; })[] | undefined; activitésPrincipales?: ("Aménagements fonciers (AFAF, remembrement)" | ... 39 more ... | "Autre")[] | undefined; }'.
  Types of property 'email' are incompatible.
    Type 'string | undefined' is not assignable to type 'string'.
      Type 'undefined' is not assignable to type 'string'.

return {

Last Updated: 22nd January at 15:37 (d287a42)

@Ynote Ynote force-pushed the ajout-filtre-activite branch from 05825ee to b9ae0b0 Compare January 21, 2025 12:15
@Ynote Ynote requested a review from DavidBruant January 21, 2025 12:20
Copy link
Collaborator

@DavidBruant DavidBruant left a comment

Choose a reason for hiding this comment

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

Trop bien !
Merci beaucoup !
C'est bon pour moi, modulo l'histoire du badge/tag

Copy link
Collaborator

@DavidBruant DavidBruant left a comment

Choose a reason for hiding this comment

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

Ah oui non, pardon !
Je crois qu'on en avait parlé à l'oral et j'ai oublié d'en reparler ici

Si on peut chercher par activité principale ici, on peut le retirer de la recherche textuelle

@Ynote Ynote requested a review from DavidBruant January 22, 2025 15:37
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