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

Gestion des erreurs TS sur les fichiers du dossier outils #91

Merged
merged 51 commits into from
Oct 14, 2024

Conversation

Ynote
Copy link
Collaborator

@Ynote Ynote commented Sep 23, 2024

J'ai commencé à faire une passe sur les fichiers du dossier outils. Il me reste un fichier à vérifier.

En parallèle, j'ai ajouté un fichier pour lancer tsc sur chaque PR ouverte en utilisant : https://github.com/marketplace/actions/run-tsc-with-reviewdog. À tester.

@@ -1,4 +1,4 @@
//@ts-check
// @ts-nocheck
Copy link
Collaborator

Choose a reason for hiding this comment

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

ooooh... c'est bien ça !!!

outils/liste-espèces.js Outdated Show resolved Hide resolved
@Ynote Ynote marked this pull request as ready for review September 26, 2024 14:34
@Ynote
Copy link
Collaborator Author

Ynote commented Sep 26, 2024

@DavidBruant Pour une raison qui m'est inconnue, je n'arrive pas à poster mes commentaires sur les fichiers de la PR. Donc je vais poster ici.

Dans le fichier sync-démarches-simplifiées-88444.js il me reste des erreurs TS. Je ne sais pas comment les gérer. En gros, quand on crée le deposant et le demandeur_personne_physique, on utilise le type PersonneInitializer mais après les infos sont remplies donc ces variables devraient pouvoir devenir des Personne. Je ne sais pas trop comment refléter ça et comment l'écrire. Je veux bien de l'aide là-dessus 🙏

@Ynote Ynote requested a review from DavidBruant September 26, 2024 14:42
Ynote added 8 commits October 10, 2024 15:32
* origin:
  "Afficher uniquement mes dossiers" (#99)
  Fix sync 2 (#97)
  Messagerie dossier en lecture (#83)
  Rendre l'outil de sync DS robuste contre le fait que la liste des espèces peut ne pas être remplie parce que question absente (#95)
@Ynote Ynote marked this pull request as draft October 12, 2024 21:37
@Ynote Ynote marked this pull request as ready for review October 12, 2024 21:47
@Ynote Ynote merged commit 1c51d5f into main Oct 14, 2024
4 checks passed
@Ynote Ynote deleted the typescript-outils branch October 14, 2024 10:39
Ynote added a commit that referenced this pull request Oct 15, 2024
…ne-saisie-espèce

* 'main' of github.com:betagouv/pitchou:
  Saisie données espèces protégées par liste d'espèces (#96)
  Gestion des erreurs TS sur les fichiers du dossier `outils` (#91)
  "Afficher uniquement mes dossiers" (#99)
  Fix sync 2 (#97)
  Messagerie dossier en lecture (#83)
dossiers.forEach(d => {
d.déposant = getPersonneId(d.déposant)
d.demandeur_personne_physique = getPersonneId(d.demandeur_personne_physique)
})
Copy link
Collaborator

Choose a reason for hiding this comment

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

Pour info et pour mémoire,
le fait d'enlever ce morceau a cassé le lien entre les dossiers et déposant/demandeur_personne_physique et donc, après une synchronisation complète tous les déposant et demandeur_personne_physique sont à null

La tentative plus bas (déposant: (déposant && déposant.id) || null,) retourne systématiquement null parce que l'objet déposant est construit à partir de DS et pas de la base de données (il n'a donc jamais de propriété déposant)

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