Skip to content
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

yform class can now use db connection 2 #1147

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

tyrant88
Copy link
Contributor

@tyrant88 tyrant88 commented Nov 6, 2021

Idee von @gharlan in #51

lib/yform.php Outdated Show resolved Hide resolved
@dergel
Copy link
Member

dergel commented Dec 13, 2021

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?

@gharlan
Copy link
Member

gharlan commented Dec 13, 2021

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.

@dergel
Copy link
Member

dergel commented Dec 13, 2021

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 :)

@tyrant88
Copy link
Contributor Author

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

@alxndr-w
Copy link
Contributor

alxndr-w commented Oct 3, 2022

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?

@dergel
Copy link
Member

dergel commented Oct 4, 2022

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.

@dgrothaus-mc dgrothaus-mc mentioned this pull request Jul 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants