Projet

Général

Profil

Bug #24495

ne pas remonter les formulaires désactivés

Ajouté par Serghei Mihai il y a presque 6 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Serghei Mihai
Catégorie:
-
Version cible:
-
Début:
13 juin 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Cela ne sert peut-être à rien de le faire dans le welco existant, mais je pose le problème quand même.

Dans la liste des formulaires qu'un agent peut saisir ceux qui sont désactivés remontent aussi. Et quand l'agent clique dessus la popup lui affiche:

error '{"err_class": "Access denied", "err_desc": "disabled form", "err": 1}'

Il faudrait ne pas afficher du tout les formulaires désactivés.

Fichiers

0001-qualification-get-only-backoffice-submission-availab.patch (3,61 ko) 0001-qualification-get-only-backoffice-submission-availab.patch Serghei Mihai, 02 juillet 2018 14:00
0001-qualification-get-only-backoffice-submission-availab.patch (6,08 ko) 0001-qualification-get-only-backoffice-submission-availab.patch Serghei Mihai, 03 juillet 2018 00:21
0001-qualification-get-only-backoffice-submission-availab.patch (6,13 ko) 0001-qualification-get-only-backoffice-submission-availab.patch Serghei Mihai, 04 juillet 2018 14:38
0001-qualification-get-only-backoffice-submission-availab.patch (6,08 ko) 0001-qualification-get-only-backoffice-submission-availab.patch Serghei Mihai, 04 juillet 2018 15:00

Révisions associées

Révision 27d62bf7 (diff)
Ajouté par Serghei Mihai il y a presque 6 ans

qualification: get only backoffice submission available forms (#24495)

Historique

#1

Mis à jour par Frédéric Péters il y a presque 6 ans

Ça me semble plutôt être l'API de w.c.s. qui ne devrait pas retourner les formulaires désactivés. (mais ça devrait déjà être le cas, ~)

C'est quelle plateforme/quelle démarche ?

#2

Mis à jour par Serghei Mihai il y a presque 6 ans

Grenoble: https://accueil-test.demarches.lametro.fr/counter/
Categorie "Economie - Recette", formulaire "Déclarer et payer ma taxe de séjour".

#3

Mis à jour par Serghei Mihai il y a presque 6 ans

L'API de wcs ne retourne pas les formulaires désactivés, sauf si elles ont une URL de redirection: https://services-test.demarches.lametro.fr/api/formdefs/

{"category": "Economie - Recette",
  "functions": {"_receiver": {"label": "Destinataire"}},
  "authentication_required": false, "description": "", 
  "title": "D\u00e9clarer et payer ma taxe de s\u00e9jour",
  "url": "https://services-test.demarches.lametro.fr/declarer-et-payer-ma-taxe-de-sejour/",
  "category_slug": "economie",
  "redirection": true,
  "keywords": [],
  "slug": "declarer-et-payer-ma-taxe-de-sejour" 
}

Welco devrait exclure les demandes faisant la redirection.

#4

Mis à jour par Serghei Mihai il y a presque 6 ans

Et donc pour que les formulaires avec rédirection ne remontent pas il faut appeler l'API avec:

?backoffice-submission=on

#5

Mis à jour par Serghei Mihai il y a presque 6 ans

Ajout du filtre ainsi que de l'email de l'agent (pour que l'API de wcs retourne les formulaires).
Testé en local et ça fonctionne.

Il manque les tests: wip.

#6

Mis à jour par Frédéric Péters il y a presque 6 ans

Entre nos briques on passe NameID, pas l'email.

#7

Mis à jour par Serghei Mihai il y a presque 6 ans

  • Fichier 0001-qualification-get-only-backoffice-submission-availab.patch ajouté

Ok.
Avec un test.

#8

Mis à jour par Serghei Mihai il y a presque 6 ans

  • Fichier 0001-qualification-get-only-backoffice-submission-availab.patch supprimé
#10

Mis à jour par Thomas Noël il y a presque 6 ans

Ajouter "user" dans la signature du __init__ (c-a-d def __init__(self, user, *args, **kwargs):) afin de ne pas avoir à faire un sale kwargs.pop

Dans le init, renommer "filters" en "params"

Dans le test, renommer SERVICES en KNOWN_SERVICES et une seule espace après les "in" (penser à régler ton emacs ;) )

Et ça m'a l'air bon à part ces petites choses.

#12

Mis à jour par Thomas Noël il y a presque 6 ans

Je pensais à un user obligatoire, en fait.

#14

Mis à jour par Thomas Noël il y a presque 6 ans

  • Statut changé de Solution proposée à Solution validée

Ack

#15

Mis à jour par Serghei Mihai il y a presque 6 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 27d62bf7ff7518033d4c63900c652715ea6229ed (HEAD -> master, origin/master, origin/HEAD)
Author: Serghei Mihai <smihai@entrouvert.com>
Date:   Tue Jun 19 11:33:04 2018 +0200

    qualification: get only backoffice submission available forms (#24495)
#16

Mis à jour par Benjamin Dauvergne il y a plus de 5 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF