payment via mobile app #17

mgmax · 2015-06-20T21:25:31Z

mgmax · 2015-06-27T14:17:59Z

Prinzipiell geht es jetzt, aber man sollte es noch ausführlich testen, insb. die oben beschriebenen doofen Fälle. Ohne echtes Gerät geht das Testen aber nicht gescheit, weil man da nicht den QRCode einlesen kann.

sedrubal · 2015-06-27T18:36:15Z

wow das geht echt gut :D

sedrubal · 2015-06-27T19:11:24Z

allerdings verwirrt der 2. QR Code und das Verhalten der App ist nicht schön, wenn man aus Versehen den unteren einscannt... Vllt. erst einblenden, wenn man nen Button drückt?

sedrubal · 2015-06-27T19:27:34Z

Wenn die Kasse aufgrund eines Fehlers abstürtzt, bekommt das die App nicht mit. Vielleicht will man auf Kurz oder Lang einen "background Worker" einbauen, der als eigenständiger Prozess kritische Tasks ordentlich verarbeitet und nicht so leicht abstürtzt, wie der Rest. Der könnte dann z.B. dem Server den Absturz melden und in der App kommt eine Erinnerung, dass man den Fehler melden will...

mgmax · 2015-06-27T20:03:01Z

Wenn die Kasse mitten im Bezahlen abstürzt, ist die Rückmeldung an die App das geringste Problem daran...

mgmax · 2015-06-28T16:22:44Z

@sedrubal der QR Dialog ist jetzt etwas schöner. Den Rest überlasse ich dir als Qt-Lernaufgabe :)

sedrubal · 2015-06-28T19:50:42Z

👍

sedrubal · 2015-07-02T21:20:46Z

App Bezahlung scheint jetzt gar nicht mehr zu gehen? Die Kasse pollt die ganze Zeit, aber bekommt keinen Warenkorb vom Server. gibt's da eig. nen Timeout, dass nicht unendlich pollt?

nightvisi0n · 2015-07-02T21:43:19Z

@sedrubal Funktioniert bei mir auch seit vorgestern nicht mehr.

mgmax · 2015-07-05T20:26:59Z

mit einer alten Version der App ca vom 27.6. (als debug build, fest auf code 1234 gestellt) geht es bei mir weiterhin. Liegt also wohl nicht an unserem Code.

mgmax · 2015-07-05T20:28:36Z

@jneureuther @sedrubal könnt ihr mal die App gegen die "Referenzimplementierung" für das Kassenterminal-API testen?
http://ec2-52-28-16-59.eu-central-1.compute.amazonaws.com/checkoutDummy/

nightvisi0n · 2015-07-05T20:53:25Z

@mgmax habe gerade die aktuelle Developmentversion[1] der App gegen die "Referenzimplementierung" getestet, funktioniert nicht.

[1] FAU-Inf2/fablab-android@dfe9ce1

mgmax · 2015-07-06T13:39:20Z

Ich hab letztens eine Fehlerbehandlung für nicht existierende Produkte ergänzt. Beim Testen sollten jetzt eigentlich (TM) keine Abstürze mehr auftreten, außer es wurde irgendwo was aus Versehen nicht abgefangen.

sedrubal · 2015-07-13T15:25:14Z

Eine falsche app URL, von der Error 404 zurück kommt wird nicht als falsche URL erkannt.

sedrubal · 2015-07-13T15:28:32Z

Wenn App Server nicht erreichbar, kommt keine Fehlermeldung
- Und beim Abbrechen hängt sich die GUI auf.

@sedrubal: Anmerkung Patrick: Kannst du bitte Schritte zum Nachvollziehen aufzeigen? Ich konnte diesen Fehler heute nicht reproduzieren.

@patkan: Schritte zum Aufhängen:

Suche IP/URL, die es nicht gibt; gerade funktionierte 123.234.165.142
In config eintragen
Import starten und abbrechen

Fehlermeldung erscheint mittlerweile im stdout, aber nicht in der GUI

(Punkte zum Abhaken hab ich oben eingetragen)

* recognize HTTP errors and return False in MobileAppCartModel.load() * ADD debug messages to JSONDecodeError and RequestError * relevant to #17

patkan · 2015-07-14T11:17:10Z

Wie will man unterscheiden zwischen zB Server kaputt und Server wartet noch?

sedrubal · 2015-07-14T11:31:19Z

Im Idealfall schon, aber dann müsste ein funktionierender Server irgendwas zurückgeben...

* recognize HTTP errors and return False in MobileAppCartModel.load() * ADD debug messages to JSONDecodeError and RequestError * improve comment on function * relevant to #17

mgmax · 2015-07-15T19:29:19Z

@sedrubal kann ich nur bedingt nachvollziehen. Wenn ich http://4.5.6.7/ als URL einstelle, braucht die HTTP-Anfrage immer 10s bis zum Timeout und währenddessen blockiert die GUI. Mit 10s Verzögerung reagiert sie aber auf Eingaben. Ich weiß nicht ob man bei cart_model.load() einen kleineren Timeout einbauen will, aber eigentlich ist das auch egal.

* recognize HTTP errors and return False in MobileAppCartModel.load() * ADD debug messages to JSONDecodeError and RequestError * improve comment on function * relevant to #17

mgmax · 2015-07-15T20:08:39Z

@patkan habe noch die Exception-Info in die Meldung dazu gepackt und es nach development gepusht

patkan · 2015-07-15T22:29:19Z

👍

sedrubal · 2015-07-16T00:03:00Z

@mgmax in den 10 Sekunden sollte die GUI nicht blockieren... Ich sag nur MVVM ;) Aber ich denke, das sind jetzt mehr Schmerzen, als dass es was bringt...

sedrubal · 2015-07-16T21:15:27Z

Negative Stückzahlen lassen das Terminal sterben.

Test-JSON:

{"cartCode":"2638286801899210886","items":[{"id":44,"productId":"9011","amount":-5.0}],"status":"PENDING","pushId":"000","sentToServer":1437080976055}

sedrubal · 2015-07-16T21:24:29Z

Dazu kommt:

Alle Antworten (http Status Code 200 aber sonst kein sinnvoller Inhalt) führen nicht dazu, dass der Vorgang abgebrochen wird
Erweiterung zu Kommentar von gerade: JSONs, die nicht valide sind, bringen das Terminal zum Absturz

ist oben in der Liste ergänzt...

(Testen kann man sowas gut, wenn man als AppServer http://0.0.0.0:8000 in die config einträgt und dann irgendwo mit python -m SimpleHTTPServer einen Webserver startet. Anhand der stdout-Ausgaben kann man in dem Ordner, wo der Server läuft, die Datei reinlegen, wonach das Terminal gerade sucht)

sedrubal · 2015-07-17T00:15:12Z

Zum Testen kann - wie gesagt - SimpleHTTPServer verwendet werden. Dazu hab ich noch dieses Skript gebaut, mit dem man schnell und einfach ein JSON mit passender cart_id / cartCode bauen kann: ./makeCart.py <id>

#!/usr/bin/env python

import json
from sys import argv
import time

id = int(argv[1])

data = {}
data["cartCode"] = id
data["items"] = []

product = {}
product["id"] = 44
product["productId"] = "9011"
product["amount"] = "5.0"

data["items"].append(product)
data["status"] = "PENDING"
data["pushId"] = "000"
data["sendToServer"] = int(time.time())

f = open(str(id), "w")
f.write(json.dumps(data))
f.close()

patkan · 2015-09-05T12:46:49Z

Für "Wenn App Server nicht erreichbar, kommt keine Fehlermeldung . " könnte man noch die Anzahl der Fehlversuche zählen und nach n abbrechen. Oder wontfix, Meinungen?

* raise Exception when no API-key is specified in config * relevant to #63 * raise Exception when num_retries times the HTTP-requests fail * relevant to #17 * catch the new Exceptions in GUI and inform the user (and abort cart import)

patkan · 2015-09-05T15:08:59Z

@mgmax, @sedrubal, @jneureuther Ich würde sagen wir sind fertig :-)
Schaut mal bitte drüber. Falls alles passt, können wir das Feature schon jetzt anschalten

mgmax self-assigned this Jun 25, 2015

mgmax added a commit that referenced this issue Jun 27, 2015

load cart from mobile app (#17)

48627f1

mgmax removed their assignment Jun 27, 2015

mgmax added a commit that referenced this issue Jun 28, 2015

load cart from app: nicer QR code dialog (#17)

60fc3af

sedrubal pushed a commit that referenced this issue Jul 2, 2015

made mobile app payment configurable (see #17)

9602fa1

sedrubal mentioned this issue Jul 6, 2015

Warenkorb Austausch geht nicht mehr FAU-Inf2/fablab-server#1

Closed

sedrubal pushed a commit that referenced this issue Jul 6, 2015

added production app server url, see #17

bcf94eb

sedrubal assigned gr7ffo and unassigned gr7ffo Jul 13, 2015

gr7ffo assigned gr7ffo and unassigned gr7ffo Jul 13, 2015

patkan added a commit that referenced this issue Jul 14, 2015

IMPROVE handling of communication with app server in case of HTTP errors

7e493f9

* recognize HTTP errors and return False in MobileAppCartModel.load() * ADD debug messages to JSONDecodeError and RequestError * relevant to #17

sedrubal pushed a commit that referenced this issue Jul 17, 2015

check json while parsing, see #17

4378f3a

mgmax mentioned this issue Jul 17, 2015

IMPROVE cart_from_app: JSON error handling #52

Closed

patkan added this to the App-Import milestone Jul 19, 2015

patkan mentioned this issue Jul 19, 2015

app-server: self-signed SSL for HTTPS #54

Closed

patkan closed this as completed Sep 7, 2015

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

payment via mobile app #17

payment via mobile app #17

mgmax commented Jun 20, 2015

mgmax commented Jun 27, 2015

sedrubal commented Jun 27, 2015

sedrubal commented Jun 27, 2015

sedrubal commented Jun 27, 2015

mgmax commented Jun 27, 2015 via email

mgmax commented Jun 28, 2015

sedrubal commented Jun 28, 2015

sedrubal commented Jul 2, 2015

nightvisi0n commented Jul 2, 2015

mgmax commented Jul 5, 2015

mgmax commented Jul 5, 2015

nightvisi0n commented Jul 5, 2015

mgmax commented Jul 6, 2015

sedrubal commented Jul 13, 2015

sedrubal commented Jul 13, 2015

patkan commented Jul 14, 2015

sedrubal commented Jul 14, 2015

mgmax commented Jul 15, 2015

mgmax commented Jul 15, 2015

patkan commented Jul 15, 2015

sedrubal commented Jul 16, 2015

sedrubal commented Jul 16, 2015

sedrubal commented Jul 16, 2015

sedrubal commented Jul 17, 2015

patkan commented Sep 5, 2015

patkan commented Sep 5, 2015

payment via mobile app #17

payment via mobile app #17

Comments

mgmax commented Jun 20, 2015

mgmax commented Jun 27, 2015

sedrubal commented Jun 27, 2015

sedrubal commented Jun 27, 2015

sedrubal commented Jun 27, 2015

mgmax commented Jun 27, 2015 via email

mgmax commented Jun 28, 2015

sedrubal commented Jun 28, 2015

sedrubal commented Jul 2, 2015

nightvisi0n commented Jul 2, 2015

mgmax commented Jul 5, 2015

mgmax commented Jul 5, 2015

nightvisi0n commented Jul 5, 2015

mgmax commented Jul 6, 2015

sedrubal commented Jul 13, 2015

sedrubal commented Jul 13, 2015

patkan commented Jul 14, 2015

sedrubal commented Jul 14, 2015

mgmax commented Jul 15, 2015

mgmax commented Jul 15, 2015

patkan commented Jul 15, 2015

sedrubal commented Jul 16, 2015

sedrubal commented Jul 16, 2015

sedrubal commented Jul 16, 2015

sedrubal commented Jul 17, 2015

patkan commented Sep 5, 2015

patkan commented Sep 5, 2015