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

IMPROVE cart_from_app: JSON error handling #52

Closed
wants to merge 6 commits into from

Conversation

mgmax
Copy link
Member

@mgmax mgmax commented Jul 17, 2015

zu #17 habe ich Bastis commit mal etwas weitergeführt.
Der Code ist noch nicht getestet, weil Feierabend :)
Mag das mal jemand kurz anschauen, minimal testen und dann mergen?

@mgmax
Copy link
Member Author

mgmax commented Jul 18, 2015

mein Code war kaputt, jetzt ist er besser + mit zuviel unittests.
Findet noch jemand einen Fehler? Sonst mache ich rebase (alles cart_from_app Zeug in 1 commit zusammenfassen) + merge.

@patkan
Copy link
Member

patkan commented Jul 19, 2015

Wie lasse ich die unittests laufen? Wenn ich einfach nur die cart_model.py ausführe, dann fehlt das shoppingbackend...

@mgmax
Copy link
Member Author

mgmax commented Jul 19, 2015

aus dem git Hauptverzeichnis: python -m FabLabKasse.shopping.cart_from_app.cart_model

@patkan
Copy link
Member

patkan commented Jul 19, 2015

Hmm, Schlaf scheint für die geistige Leistung zuträglich zu sein :-)
Bisher habe ich keine Fehler gefunden. Anderes Thema, aber das könnte man auch mal andenken, ist, wie man halbwegs automagisch alle vorhandenen unittests durchorgeln kann.

@patkan patkan added this to the App-Import milestone Jul 19, 2015
@mgmax
Copy link
Member Author

mgmax commented Jul 19, 2015

fertig

@mgmax mgmax closed this Jul 19, 2015
@mgmax mgmax deleted the cart-from-app-json-exceptions branch July 19, 2015 20:03
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