Skip to content
This repository has been archived by the owner on Jun 13, 2020. It is now read-only.

Refactor names of database tables and entities #87

Open
chaoran-chen opened this issue Apr 10, 2020 · 5 comments
Open

Refactor names of database tables and entities #87

chaoran-chen opened this issue Apr 10, 2020 · 5 comments
Milestone

Comments

@chaoran-chen
Copy link
Member

chaoran-chen commented Apr 10, 2020

The database tables device, consumable, and personal should be prefixed with offer_.

And maybe also change personal / manpower to personnel.

@Timo-Weike
Copy link
Member

Man könnte dann auch überlegen ob man im gleichen zuge dIe Namen an C# Names Koventionen anpasst. (Also CamalCase und ohne _, oder spricht auf seiten von Postgress was dagegen?

@chaoran-chen
Copy link
Member Author

chaoran-chen commented Apr 11, 2020

Gute Idee. Ich fände es gut, wenn im Code die C# Namenskonvention komplett eingehalten wird.

Gleichzeitig würde ich gerne in der Datenbank übliche SQL-Namen (mit _ und kleingeschrieben) verwenden. Aber dazu gibt es im EntityFramework Mappingsannotationen:
https://stackoverflow.com/questions/42190909/how-to-specify-entity-framework-core-table-mapping

@MaxWies
Copy link
Contributor

MaxWies commented Apr 11, 2020

Wenn es das Mapping so gibt, dann ist CamelCase sehr sinnvoll im Backend. In Postgres sollten wir auf Großbuchstaben verzichten.

@MaxWies
Copy link
Contributor

MaxWies commented Apr 11, 2020

Wir könnten personal mehr zu einem Berufsbegriff machen. Wie wäre es mit labour ?

@chaoran-chen
Copy link
Member Author

Oder staff? xD Im Frontend wird aktuell im Code einheitlich personnel verwendet.

@chaoran-chen chaoran-chen added this to the release-1.4 milestone Apr 12, 2020
@chaoran-chen chaoran-chen removed this from the release-1.5 milestone Apr 20, 2020
@chaoran-chen chaoran-chen added this to the release-1.5 milestone Apr 26, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants