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

fix: Korrektur von Auskunftsskripten im Rahmen der GID7-Umstellung (NDS) #2

Merged
merged 3 commits into from
Feb 28, 2024

Conversation

colaheart
Copy link

Dieser PR löst einige Fehler, die bei Tests mit GID7-Testdaten des LGLN (NDS) mit den bisherigen Auskunftsskripten auffielen. Ziel ist, die Fehler zu bereinigen und den Funktionsumfang soweit möglich nicht zu verändern.

@colaheart
Copy link
Author

PR ist noch als Draft markiert, um etwaige weitere Lösungen für #3 zu ergänzen.

@tschuettenberg tschuettenberg marked this pull request as ready for review January 31, 2024 10:34
@tschuettenberg
Copy link

(die Markierung als "ready for review" war unabsichtlich, bei der Suche danach wie man in eine PR pusht...)

@tschuettenberg
Copy link

In Verbindung mit der Entfernung von alkis_elemente (vgl. #3 (comment) ) funktioniert diese Korrektur #2 bei mir - bis auf die Gebäude (alkishaus.php).

@colaheart colaheart marked this pull request as draft February 6, 2024 14:20
@colaheart colaheart force-pushed the fix/gid7-database-queries-nds branch from 0442bd7 to c659bad Compare February 7, 2024 15:43
@colaheart colaheart marked this pull request as ready for review February 7, 2024 15:44
@colaheart colaheart force-pushed the fix/gid7-database-queries-nds branch from c659bad to 7b9c86a Compare February 7, 2024 15:56
@wirkus wirkus self-requested a review February 8, 2024 12:54
Copy link

@wirkus wirkus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ich habe die Änderungen in alkisfsnw.php und alkishaus.php mit den Demodaten aus Bayer getestet, es funktioniert.

@colaheart
Copy link
Author

Da wir auch die ersten Datensätze nach dem neuen Schema erhalten haben ist dieser Stand bei uns aktiv im Einsatz.

@astroidex Von dir wurde ja ein Review angefragt. Willst/kannst du drüber schauen oder soll jemand anderes als zweite Person drüber schauen?

Wenn zwei/drei Anwender ihr OK geben ist mein Vorschlag für einen ersten GID7-Stand zu mergen.

@astroidex
Copy link
Contributor

@colaheart ja, denke es könnte noch eine weitere Person draufschauen. @tschuettenberg hast du Lust/Zeit?

@tschuettenberg
Copy link

Hallo, ich habe nochmal alles geprüft und die Änderungen durch die drei Commits funktionieren nach wie vor bei mir, d.h. mit Produktivdaten aus NRW, importiert mit norGIS ALKIS-Import 4.1-6.

@astroidex
Copy link
Contributor

astroidex commented Feb 28, 2024

Ich würde den Code nun mergen. @wirkus und @tschuettenberg haben getestet.

@astroidex astroidex merged commit b2f2f5f into gid7 Feb 28, 2024
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