smart-village-solutions / smart-village-app-cms

A CMS based on JSON schema of smart-village api
GNU General Public License v3.0
7 stars 2 forks source link

Update records instead of deleting and creating #128

Closed donni106 closed 2 years ago

donni106 commented 2 years ago

based on main-server PR: https://github.com/ikuseiGmbH/smart-village-app-mainserver/pull/210

Bildschirmfoto 2021-11-18 um 18 48 04

SVA-176

marcometz commented 2 years ago

Woher weiß das CMS ob der Server Update statt Create/Delete für die Resource unterstützt? Muss das CMS das wissen, oder sind alle Mainserver und alle CMS immer identisch?

donni106 commented 2 years ago

Woher weiß das CMS ob der Server Update statt Create/Delete für die Resource unterstützt? Muss das CMS das wissen, oder sind alle Mainserver und alle CMS immer identisch?

hmmmm. gute frage. das liegt in unserer hand. wir könnten dafür sorgen, dass alles "gleichzeitig" online geht. am besten zu phasen, wo keine imports stattfinden? oder wir brauchen ab sofort eine versionierung und alte sowie neue varianten im code?

donni106 commented 2 years ago

Woher weiß das CMS ob der Server Update statt Create/Delete für die Resource unterstützt? Muss das CMS das wissen, oder sind alle Mainserver und alle CMS immer identisch?

hmmmm. gute frage. das liegt in unserer hand. wir könnten dafür sorgen, dass alles "gleichzeitig" online geht. am besten zu phasen, wo keine imports stattfinden? oder wir brauchen ab sofort eine versionierung und alte sowie neue varianten im code?

Wir haben bei Absprachen keine Besonderheiten festgestellt. Generell sollten wir aber Main-Server vor CMS deployen, da CMS-Änderungen meistens auf Main-Server-Änderungen aufsetzen.

codeclimate[bot] commented 2 years ago

Code Climate has analyzed commit 58ad87cb and detected 28 issues on this pull request.

Here's the issue category breakdown:

Category Count
Duplication 10
Security 6
Complexity 12

View more on Code Climate.