-
-
Notifications
You must be signed in to change notification settings - Fork 24
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
adds endpoint for retrieving email-templates incl. placeholders #2623
Conversation
Ich habe für #2605 keinen neuen Branch erstellt und den Commit hier hineingepackt. |
@j-dimension den Commit 80300c9 habe ich nicht absichtlich gemacht. Ist das ein Problem, wenn NetBeans mich beim commiten/pushen fragt, ob ich den branch mergen will? Ist der Commit "Merge origin/master into feat/add_endpoints_emailtemplates" hier völlig verkehrt? Die anderen beiden Enhancements waren naürlich Absicht :) |
Ich bin hier ein wenig lost. Ich weiß nicht genau wie die Thunderbird-Integration die Endpunkte orchestrieren würde - mir erscheint Folgendes sinnvoll:
Die Änderungen hier bewirken, dass beim Auflisten der Vorlagen bereits alle Vorlagen ausgefüllt werden, das kann eine ziemliche Datenbankabfragemenge in kürzester Zeit generieren. Das Ersetzen der Platzhalter ist Code der bereits vorhanden ist, lass uns den nachnutzen. @iradraconis darf ich Hand anlegen? |
@j-dimension ja, du hast absolut recht. Ich habe nicht bedacht. Wie du geschrieben hast, macht es mehr Sinn. Du kannst es gerne anpassen. |
Mir wäre jedenfalls wichtig, eine Funktionalität ähnlich der Beteiligten Matrix am Ende gewährleisten zu können. Deswegen hatte ich die vollständigen Vorlagen einschließlich Platzhalter und deren Werte aufgenommen. |
- GET /email/templates returns list of available templates - GET /email/{caseId}/{templateName} returns template with filled placeholders Refactor email template endpoints: - Split existing getEmailTemplates into two dedicated endpoints - Simplify template list to return only template names - Improve response structure by including placeholder values separately
- Document template list endpoint with response codes - Document template detail endpoint with parameters and responses
- added support for forms placeholders - added mimetype to output, so that a client can determine whether it deals with plain text or HTML - refactored code to use existing logic / avoid duplication - issue #2623
Ich hatte hier in einem Comit noch im Responste die Originalvorlage mit Platzhaltern (nicht ausgefüllt). Das hast, du glaube ich, wieder entfernt. Das wäre nützlich, um die Platzhalter durch andere Beteiligte ausfüllen zu lassen für den Fall, dass es 2 x Dritte in der Akte gibt oder mehr als 1 Gericht.... |
completes #2055
Ich hoffe, ich habe hier nichts kaputt implementiert.
Response enthält: