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

fix pour les uri des theses en preparation consultées par l'API ( de la forme /api/v1/theses/these/s383095 étaient ignorées) + modifs commentaires exemples #851

Merged
merged 6 commits into from
Sep 26, 2024

Conversation

ThomasMic
Copy link
Contributor

nous proposons cette modification qui était en attente ( à cause d'un autre commit bloquant pour nous voir #850 )

@felixleo22
Copy link
Contributor

Est-ce que vous avez fait des mises à jour sur le fichier de test ? si c'est le cas, penser à le commit que je puisse voir pourquoi les tests ne passent pas. Si vous me dite que le fichier est bien à jour, je vous proposerai des modifications pour qu'ils passent

@ThomasMic
Copy link
Contributor Author

Bonjour, alors après discussion avec Frédéric hier :

  • c'est la dernière ligne qui ne devrait pas être dans le fichier de test (
    ;;;SEARCH;HTML;;;https://theses.fr/api/v1/theses/recherche/?q=test&debut=0&nombre=10&tri=pertinence;200
    ) car elle ne correspond à rien au niveau de ce que gère le parser
  • il semblerait qu'on est pas encore dans un cycle "classique" ( Analogist->génération du fichier de test) car c'est la première fois que nous (ABES) soumettons une demande vers Analogist
  • le fichier de test actuel a été élaboré "avant" que l'ABES commence à travailler sur ezpaarse
  • pour l'instant, soit je peux (autorisations ?) moi même supprimer la dernière ligne du fichier de test pour qu'il passe, soit n'hésitez pas à le faire directement (je ne sais pas si je peux "modifier" ma PR d'ailleurs ?)

Merci

@felixleo22
Copy link
Contributor

Oui pas de soucis, vous pouvez modifier la pull request en ajoutant des commits, on reportera sur Analogist le contenu du fichier de test une fois vos développements terminé.

this URI is not managed by parser's code
align test values to functionnal changes
@ThomasMic
Copy link
Contributor Author

C'est fait, tests OK

@felixleo22 felixleo22 merged commit 3c89d14 into ezpaarse-project:master Sep 26, 2024
2 checks passed
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