yform icon indicating copy to clipboard operation
yform copied to clipboard

yform class can now use db connection 2

Open tyrant88 opened this issue 3 years ago • 6 comments

Idee von @gharlan in #51

tyrant88 avatar Nov 06 '21 01:11 tyrant88

Danke für die Geduld, leider habe ich mich hier nicht ran getraut, weil ich nicht glaube, dass es so funktioniert. Was soll gelöst werden? Die Schemata in der DB1 und die andere/manche Tabelle/n z.B. in der db2?

dergel avatar Dec 13 '21 00:12 dergel

Der PR hier setzt meinen Vorschlag für einen ersten Schritt (siehe Issue) um. Er lässt also bewusst den Manager noch ganz außen vor, und bringt erstmal nur die Funktionalität in die Basis-yform.

gharlan avatar Dec 13 '21 00:12 gharlan

ok, sorry. habe ich leider nicht genau genug gelesen. @tyrant88 wenn du schauen könntest, was noch angepasst werden müsste, damit es keinen Konflikt gibt und noch mal checkst, ob es mit den aktuellsten Änderungen passt, übernehme ich es gerne. Ich werde es nicht testen, deswegen bitte unbedingt selbst alle Fälle testen - Danke :)

dergel avatar Dec 13 '21 11:12 dergel

Alright. Jetzt habe ich wirklich alle betroffenen values getestet. Wenn ich lokal bei mir mit dem aktuellen master merge, gibt es keine Konflikte.

tyrant88 avatar Dec 18 '21 01:12 tyrant88

damit es keinen Konflikt gibt und noch mal checkst, ob es mit den aktuellsten Änderungen passt, übernehme ich es gerne.

Alright. Jetzt habe ich wirklich alle betroffenen values getestet.

Kann man noch etwas dafür tun, das die Annahme des PRs begünstigt?

alxndr-w avatar Oct 03 '22 21:10 alxndr-w

Habe ich hier etwas vernachlässigt. Soweit ich das verstehe gehts hier nicht um YOrm und be Sachen, sondern ausschließlich um Formulare. Das wäre toll, wenn das noch um PR stehe würde und am besten auch direkt in die Doku einfließt. Es gibt noch in paar Konflikte, die gelöst werden müssten.

dergel avatar Oct 04 '22 09:10 dergel