Development #25928
retirer la page intermédiaire, directement afficher les transactions
Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
29 août 2018
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Description
Aujourd'hui quand on va dans la section "paiement en ligne" on a une page intermédiaire proposant d'aller soit vers la liste des régies soit vers la liste des transactions; je supprimerais cette page intermédiaire pour afficher directement les transactions, avec un lien pour aller dans la configuration des régies.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Fichier 0001-lingo-change-online-payment-page-to-directly-display.patch 0001-lingo-change-online-payment-page-to-directly-display.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Serghei Mihai il y a plus de 5 ans
- Statut changé de Solution proposée à Solution validée
Le patch s'applique pas, il dépend d'un autre ?
Mis à jour par Serghei Mihai il y a plus de 5 ans
- Statut changé de Solution validée à Solution proposée
Il faut mettre à jour les tests:
def test_transactions_search(app, admin_user):
for i in range(50):
Transaction(status=eopayment.PAID,
order_id='order id %s' % (i+1),
bank_transaction_id='bank id %s' % (i+1),
amount=1+i).save()
app = login(app)
> resp = app.get('/manage/lingo/transactions/', status=200)
tests/test_lingo_manager.py:142:
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _
/tmp/tox-serghei/combo/py3-django111/local/lib/python2.7/site-packages/django_webtest/__init__.py:124: in get
response = super(DjangoTestApp, self).get(url, **kwargs)
/tmp/tox-serghei/combo/py3-django111/local/lib/python2.7/site-packages/webtest/app.py:335: in get
expect_errors=expect_errors)
/tmp/tox-serghei/combo/py3-django111/local/lib/python2.7/site-packages/django_webtest/__init__.py:87: in do_request
expect_errors)
/tmp/tox-serghei/combo/py3-django111/local/lib/python2.7/site-packages/webtest/app.py:655: in do_request
self._check_status(status, res)
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _
self = <django_webtest.DjangoTestApp object at 0x7fd106378550>, status = 200, res = <404 Not Found text/html body='<!DOCTYPE...l>\n'/793>
def _check_status(self, status, res):
if status == '*':
return
res_status = res.status
if (isinstance(status, string_types) and '*' in status):
if re.match(fnmatch.translate(status), res_status, re.I):
return
if isinstance(status, string_types):
if status == res_status:
return
if isinstance(status, (list, tuple)):
if res.status_int not in status:
raise AppError(
"Bad response: %s (not one of %s for %s)\n%s",
res_status, ', '.join(map(str, status)),
res.request.url, res)
return
if status is None:
if res.status_int >= 200 and res.status_int < 400:
return
raise AppError(
"Bad response: %s (not 200 OK or 3xx redirect for %s)\n%s",
res_status, res.request.url,
res)
if status != res.status_int:
raise AppError(
> "Bad response: %s (not %s)\n%s", res_status, status, res)
E AppError: Bad response: 404 Not Found (not 200)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Solution proposée à Résolu (à déployer)
Avec l'adaptation au test :
commit 698253a1492ab707e3ce8cbdbeaa2c7d16386a9e
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Sun Aug 19 19:16:31 2018 +0200
lingo: change online payment page to directly display transactions (#25928)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
lingo: change online payment page to directly display transactions (#25928)