search

yakamara / yform

YForm für REDAXO 5 – Formulare im Frontend und Backend mit Verwaltung von Datenbank-Tabellen.
MIT License
77 stars 55 forks source link

E-Mail Templates durchsuchbar machen .. #406

Open dergel opened 6 years ago

tbaddade commented 6 years ago

auch via Watson?

dergel commented 6 years ago

wäre cool :)

alxndr-w commented 4 years ago

@tbaddade sehe ich das richtig, man müsste das hier auf die E-Mail-Templates-Tabelle übertragen?

https://github.com/FriendsOfREDAXO/sked/tree/master/lib/watson

tbaddade commented 4 years ago

https://github.com/FriendsOfREDAXO/sked/tree/master/lib/watson

Du bist im falschen Repo … Das ist Sked

Die Anpassungen gehören momentan für YForm hier hin https://github.com/tbaddade/redaxo_watson/tree/master/lib/Watson/Workflows/YForm

Dazu müsste es eine Class YFormEmailTemplateSearch erstellt werden und diese im Provider registriert werden.

alxndr-w commented 4 years ago

Du bist im falschen Repo … Das ist Sked

Nene, ich war schon richtig ;) Ich hätte mir das von Sked abgeschaut und hier in YForm eingebaut.

Aber jetzt sehe ich, dass Watson schon was für YForm mitbringt und damit ist mir auch klarer, dass mein Ansatz verkehrt ist.

Sollen wir dann das Issue hier zu machen und bei dir drüben auf?

tbaddade commented 4 years ago

Sollen wir dann das Issue hier zu machen und bei dir drüben auf?

Nein, da der Workflow eigentlich zu YForm gehört. Watson sollte max. die Core-AddOns mitbringen. Das ist halt noch eine grundlegende Entscheidung. Will Watson YForm durchsuchen oder will YForm Watson nutzen um durchsucht zu werden und Watson selbst nur die API bereitstellt?

alxndr-w commented 4 years ago

Ich habe einen Entwurf hierzu umgesetzt und bitte um Feedback. Wenn ich glaube, verstanden zu haben, wie das funktioniert, würde ich gerne noch weitere Addons an Watson anbinden. 💙 @tbaddade

tbaddade commented 4 years ago

Danke aber

Das ist halt noch eine grundlegende Entscheidung.

Ich würde das gern erst einmal abwarten. Daraus folgt dann wo die Suche zum Schluss integriert wird.