search

ADFC-Hamburg / adfc-t30-api

MIT License
0 stars 0 forks source link

Forderungsmail abschicken #71

Closed bensteffen closed 4 years ago

bensteffen commented 4 years ago

Zum Absenden der Forderungsmail gibt es jetzt folgenden API-Call:

GET http://localhost/adfc/adfc-t30-api/api/demand.php?emailId=<emailId>

wobei <emailId> die ID der Forderungs-Email ist.

Es wird überprüft, ob die Email für den angemeldeten Nutzer existiert und ob für den der Mail zugeordneten Straßenabschnitt schon eine Mail verschickt wurde (über das Feld mail_sent in demandedstreetsection).

Die Mail-Adresse des zuständigen PKs wird über die PK-Datenbanktabelle ermittelt.

Der Absender wird in CC gesetzt

Die echte PK-Adresse wird durch eine Test-Adresse ersetzt.

Dazu gehören in FlexAPI folgende kleine PRs: https://github.com/ADFC-Hamburg/flexapi/pull/39 https://github.com/ADFC-Hamburg/flexapi/pull/40

bensteffen commented 4 years ago

Folgende Punkte fehlen noch:

image

image

Hier die gelb markierten: image

tabacha commented 4 years ago

Ich hatte daraus später extra mehrere Issues gemacht.

Kümmer dich bitte zuerst nur um Issue: #56 und danach um #57, #64, #65. Ich hatte da noch diverse Details ergäntzt korrigiert.

Folgende Punkte fehlen noch:

image

image

Hier die gelb markierten: image

tabacha commented 4 years ago

Hab noch ein paar kleinigkeiten auf dem Feature-Branch ergänzt.

TODO: Der Absender soll nicht die globale E-Mail aus den Settings sein, sondern die Person die das angestozen hat selber.

genial ist, dass ich jetzt weiter das Frontend entwickeln kann. :-)

tabacha commented 4 years ago

Ach und bitte bei CC noch die defaultFrom aus der api.conf.php mit aufnehmen (Wiebke möchte die Mails als CC).

bensteffen commented 4 years ago

Frage zu

TODO: Der Absender soll nicht die globale E-Mail aus den Settings sein, sondern die Person die das angestozen hat selber.

Ist das technisch überhaupt möglich? Tatsächlich habe ich es probiert als Absende-Adresse die Mail-Adresse des Users einzustellen. Der Versand der Email schlug dann mit Exception fehl, weil die Absender-Adresse gleich der Adresse in den SMTP-Credentials sein muss. Ist ja auch klar irgendwie: Wir können doch nicht mit unserem SMTP-Servers im Namen eines anderen STMP-Servers (den des Users) Mails verschicken. Oder übersehe ich etwas?

tabacha commented 4 years ago

Ja es ist technisch möglich, ansonsten würden z. B Mailinglisten nicht funktionieren. Evtl verhindert den Versand dein ISP, aber das hat nichts mit dem was technisch möglich ist.

Wir können es in jedem Fall vom Tools Server.

Gruß Sven

Am 7. Oktober 2019 16:54:36 schrieb bensteffen notifications@github.com:

Frage zu TODO: Der Absender soll nicht die globale E-Mail aus den Settings sein, sondern die Person die das angestozen hat selber. Ist das technisch überhaupt möglich? Tatsächlich habe ich es probiert als Absende-Adresse die Mail-Adresse des Users einzustellen. Der Versand der Email schlug dann mit Exception fehl, weil die Absender-Adresse gleich der Adresse in den SMTP-Credentials sein muss. Ist ja auch klar irgendwie: Wir können doch nicht mit unserem SMTP-Servers im Namen eines anderen STMP-Servers (den des Users) Mails verschicken. Oder übersehe ich etwas? — You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.

bensteffen commented 4 years ago

Erledigt:

Die weiteren Nutzer die angefangen haben eine E-Mail zu schreiben darüber informiert werden, dass jemand anders schneller war

TODO: Der Absender soll nicht die globale E-Mail aus den Settings sein, sondern die Person die das angestozen hat selber.

Ob sich die Absende-Adresse von der SMTP-Anmelde-Adresse unterscheiden darf, hängt wohl anscheinend vom jeweiligen SMTP-Server ab, der verwendet wird... Bei meinem ist es wohl ein Problem.

tabacha commented 4 years ago

TODO: Der Absender soll nicht die globale E-Mail aus den Settings sein, sondern die Person die das angestozen hat selber.

Ob sich die Absende-Adresse von der SMTP-Anmelde-Adresse unterscheiden darf, hängt wohl anscheinend vom jeweiligen SMTP-Server ab, der verwendet wird... Bei meinem ist es wohl ein Problem.

Wie wollen wir da weiter vorgehen. Sollen wir den PR Mergen und du machst dafür einen neuen? Brauchst du einen Testaccount auf einem Server wo es geht, oder willst du das blind machen?

bensteffen commented 4 years ago

Ich hab es jetzt erst mal über eine Abfrage der Umgebung (FlexAPI::$env; ist bei mir lokal auf 'def' gestellt) gelöst. Aber ein Test-Account wäre natürlich die beste Lösung.