Skip to content
This repository has been archived by the owner on Jan 19, 2019. It is now read-only.

Login pizza.de-Account #11

Open
Lem opened this issue Mar 13, 2014 · 2 comments
Open

Login pizza.de-Account #11

Lem opened this issue Mar 13, 2014 · 2 comments

Comments

@Lem
Copy link

Lem commented Mar 13, 2014

Hi,
sehr toll, danke erstmal.

Ich hab nur ein Problem: Wenn ich eine Bestellung aufgeben will, kann ich mich nicht in mein Pizza.de-account einloggen und somit nicht via paypal zahlen.

Gibt es eine Chance dass dies später supportet wird?

@breunigs
Copy link
Owner

Ich würde es nicht ausschließen, allerdings wird der Patch nicht von mir kommen. Das Problem ist, das ich keinen PayPal Account habe und mir auch keinen zulegen möchte. Allerdings ist eine Bestellung durchführen die einzige Möglichkeit die Funktionalität zu testen. Fake-Bestellungen bei einem befreundeten Pizza-Service sind daher schwierig, da man ja wirklich Geld überweisen muss… vermute ich zumindest, wie funktioniert das denn genau, wenn man einen pizza.de-Account hat?

Pizza.de will auch keine API bereitstellen, d.h. ich sehe auch keine Chancen das man von dieser Seite Hilfe bekommt.

Falls Du es implementieren möchtest: ein guter Anfang ist der knddomain=1 URL-Parameter. Der sorgt dafür, dass das meiste vom Pizza.de-Branding verschwindet. Außerdem nimmt sich Pizza.de einen größeren Anteil, wenn man einen Shop über Pizza.de gefunden hat, statt über die eigene Webseite des Pizza-Services. Da Pizza.de keine Gruppenbestellungs-Funktion implementiert hat, würde ich darauf bestehen, dass dieser Parameter immer anhängt, außer jemand will sich wirklich bei Pizza.de anmelden.

@breunigs
Copy link
Owner

6db2f71 implements the last piece that makes a login technically possible on pizza.de. Be aware that HipsterPizza will MITM the login on pizza.de.

There are remaining issues:

  • the logged in status is not (and should not) be shared between participants
    • everyone will see the full price, without discount
    • the order list will show the full price, without discount
    • replaying directly on the "logged in" shop page warns about the price difference, because discount is not taken into account
  • basket submission currently loads the "logged out" shop page. You manually have to remove &knddomain=1 from the URL for it to detect your login.
  • logging in on the "logged out" shop page is untested.
  • submitting an order logged in is untested.
  • caching is basically disabled when someone is logged in, making HipsterPizza slow

I welcome suggestions on how to handle the discount/pricing display issue.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants