Notifications emails - Githubissues

keboola / app-orchestrator-trigger

MIT License

0 stars 1 forks source link

Notifications emails #4

Open ErikZigo opened 6 years ago

ErikZigo commented 6 years ago

Pridat novy parametr pro nastaveni emailovych notifikaci orchestratoru.

Ten se pak bude posilat na API orchestratoru pri zakladani noveho jobu

[ ] Proverit jak se chova json form v UI po pridani noveho parametru
[x] Proverit jak presne funguje validace povinnych polozek v UI

ErikZigo commented 5 years ago

Zda se ze validace v json schema formulari jsou rozbity https://github.com/keboola/kbc-ui/pull/2261

V UI musi byt notifikacni emaily jako povinna polozka (jinak se u nas nevykresli input). Na strane aplikace bych to udelal jako volitelny parametr, alespon to bude zpetne kompatibilni.

davidesner commented 5 years ago

Ahoj, nechteli byste tohle mergnout? Bylo by to super. Vim, ze to za chvilku nebude aktualni, ale klientovi na tom stoji cely flow a bohuzel ted nedostavaji notifikace. Nez probehne migrace na novy Orchestrator tak to porad bude nejaky tyden trvat, ne-li dele. Klidne opravim to schema. Pokud to neplanujete, tak bych si to stejne asi forknul a docasne hodil docasne jako svoji komponentu at jsou happy. Diky!