BP1/BP2 SQL procedures and process-based structure#63
Conversation
BP1: Procedure fuer kritische Ressourcen erstellt. BP1: Procedure fuer ablaufende Ressourcen erstellt. BP1: Procedure fuer Nachschubanforderung erstellt. SQL-Spiegel und Dokumentation fuer Datenbankabfragen, BPMN, Lastenheft/Pflichtenheft und SQL-Uebersicht angepasst.
Ph4ntomic
changed the title
CyZeTLC
left a comment
CyZeTLC
left a comment
There was a problem hiding this comment.
Die restApi Logik passt so nicht, da wir auf Prisma umstellen & die bereits aktuelle Version routing beinhaltet.
CyZeTLC
left a comment
CyZeTLC
left a comment
There was a problem hiding this comment.
Am besten überlass die API-Logik mir, dann kommen da keine komischen Änderungen rein, welche nicht ganz zum System passen.
Revert the restApi.php SQL path-routing changes from PR CyZeTLC#63 per review feedback. Keep the BP1/BP2 SQL organization, stored procedures, docs, and related frontend references intact.
|
Danke fürs Feedback 💯 |
CyZeTLC
left a comment
CyZeTLC
left a comment
There was a problem hiding this comment.
Wäre top, wenn du die Änderungen in den TypeScript Datein auch noch raus nehmen könntest. Ansonsten funktioniert die RestAPI garnicht mehr.
|
Die TypeScript-Anpassungen sind wie gewünscht zurückgebaut, damit die bestehende REST-API unverändert bleibt. Als kurzer Reminder: Die SQL-Dateien und Stored Procedures sind weiterhin nach den besprochenen Bereichen Da die Datenbanklogik nach der besprochenen Vorgabe über Stored Procedures laufen soll, sind bei der späteren Prisma-/API-Anbindung vor allem die fachliche Zuordnung, die Procedure-Namen und die Ablagepfade der Stored-Procedure-Dateien relevant. Die API-/Prisma-Anbindung wird in diesem PR selbst nicht geändert. Bitte nochmal prüfen. Wenn noch etwas auffällt, schreib einfach Schönen Sonntag noch :-) |
2 participants
Zusammenfassung:
Diese Pull Request bündelt die vollständige BP1/BP2-Datenbankarbeit in einem zusammenhängenden Review.
sql/queries/{bp1,bp2,shared,general}undsql/storedProcedure/{bp1,bp2,shared,general}.sql/sqlOverview.txt.Warum:
Die vorherige SQL-Ordnerstruktur hat prozessspezifische Abfragen, allgemeine Abfragen und Stored Procedures vermischt. Die neue Struktur macht klar sichtbar, welche Dateien zu BP1, BP2, gemeinsamen Ressourcenabfragen oder allgemeinen WebApp-Abfragen gehören.
Enthaltene Commits:
6aba52aBP1: add procedures for critical resources and resupplyb9d78d0BP2: add procedures and organize SQL by processa08ebbfchore: reorganize SQL structure by query type and processPrüfung:
../sql/get...,sql/get...,sql/bp1,sql/bp2undsql/sharedwurden geprüft und in aktiven API-/Frontend-/Doku-Pfaden bereinigt.:-)