Project

General

Profile

Development #69108

api: performance du endpoint list, permettre d'exclure les evolutions

Added by Benjamin Dauvergne 5 months ago. Updated 3 months ago.

Status:
Fermé
Priority:
Normal
Target version:
-
Start date:
15 September 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

Cf. les résultats cProfile dans #69090

Les évolutions ne sont pas généralement utiles il faudrait découper full=on en condition plus fine comme include-evolution, include-fields, include-roles, etc..


Files

0001-api-split-full-param-69108.patch (17.8 KB) 0001-api-split-full-param-69108.patch Lauréline Guérin, 08 November 2022 04:25 PM

Related issues

Related to w.c.s. - Development #69090: Requête web beaucoup trop longue sur /api/cardsFermé14 September 2022

Actions
Related to w.c.s. - Development #69109: api: performance du endpoint list, charger les parts de manière paresseuseFermé15 September 2022

Actions
Related to w.c.s. - Bug #71303: API carddata avec "include-evolution=on" seulement, erreur 500Fermé15 November 2022

Actions
Related to Combo - Development #71328: Cellule Fiche(s): au lieu du param full=on, passer les param include-fields=on include-submission=on et include-workflow=onFermé15 November 2022

Actions
Related to w.c.s. - Development #71330: api formdata: ajouter un param include-workflow-dataFermé15 November 2022

Actions

Associated revisions

Revision 29f33529 (diff)
Added by Lauréline Guérin 3 months ago

api: split full param (#69108)

new params:
- include-fields
- include-evolution
- include-roles
- include-submission
- include-workflow

History

#1

Updated by Benjamin Dauvergne 5 months ago

#2

Updated by Benjamin Dauvergne 5 months ago

  • Blocked by Development #69109: api: performance du endpoint list, charger les parts de manière paresseuse added
#3

Updated by Frédéric Péters 5 months ago

  • Blocked by deleted (Development #69109: api: performance du endpoint list, charger les parts de manière paresseuse)
#4

Updated by Frédéric Péters 5 months ago

  • Related to Development #69109: api: performance du endpoint list, charger les parts de manière paresseuse added
#5

Updated by Lauréline Guérin 3 months ago

#6

Updated by Frédéric Péters 3 months ago

        data['criticality_level'] = self.criticality_level
        data['api_url'] = self.get_api_url()
        data['backoffice_url'] = self.get_backoffice_url()

pourraient rester tout le temps là, non ? (plutôt qu'uniquement si include_fields)

#7

Updated by Lauréline Guérin 3 months ago

Ce sont les items que je ne sais pas ranger dans les options evolution, roles, workflow, etc; ils ne sont pas renvoyés en mode full=off, c'était pour avoir un résultat qui ressemble.

#8

Updated by Frédéric Péters 3 months ago

  • Status changed from Solution proposée to Solution validée

Avec idée à un moment de pouvoir éliminer la condition if include_fields or include_evolution or include_roles or include_submission or include_workflow: et toujours appeler get_json_export_dict ? ça me va. (et donc ok aussi pour laisser en l'état pour le moment et quand on voudra l'éliminer on pourra mesurer les perfs).

#9

Updated by Lauréline Guérin 3 months ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 29f33529786dec74bd08c95090ba3b0e7f942d3d
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date:   Tue Nov 8 13:49:27 2022 +0100

    api: split full param (#69108)

    new params:
    - include-fields
    - include-evolution
    - include-roles
    - include-submission
    - include-workflow
#10

Updated by Lauréline Guérin 3 months ago

  • Related to Bug #71303: API carddata avec "include-evolution=on" seulement, erreur 500 added
#11

Updated by Lauréline Guérin 3 months ago

  • Related to Development #71328: Cellule Fiche(s): au lieu du param full=on, passer les param include-fields=on include-submission=on et include-workflow=on added
#13

Updated by Lauréline Guérin 3 months ago

  • Related to Development #71330: api formdata: ajouter un param include-workflow-data added
#14

Updated by Transition automatique 3 months ago

  • Status changed from Résolu (à déployer) to Solution déployée
#15

Updated by Transition automatique 21 days ago

Automatic expiration

Also available in: Atom PDF