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

.gitignore in ER diagram #2

Open
jaanos opened this issue May 13, 2019 · 0 comments
Open

.gitignore in ER diagram #2

jaanos opened this issue May 13, 2019 · 0 comments

Comments

@jaanos
Copy link

jaanos commented May 13, 2019

Vidim, da imate na repozitoriju datoteko auth.py z geslom, ki je tako za vekomaj objavljeno na internetu. Tako svetujem, da @MatijaGH zamenjaš svoje geslo, datoteko auth.py pa pobrišete iz repozitorija - kot rečeno v #1, naj vsak pri sebi poskrbi za svojo kopijo s svojimi podatki za prijavo - poglejte si pri pdurcik/Baza-clanstva-RDR#3, kako lahko poskrbite za ustrezne pravice dostopa (aplikacija se bo prijavila kot javnost, tako da naj namesto auth uvozi auth_public).

V .gitignore že imate vnos auth.py, tako da po brisanju te datoteke ne boste pomotoma znova dodali. Svetujem, da tja dodaste tudi vrstico __pycache__ ter to mapo pobrišete iz repozitorija - gre namreč za prevedene Pythonove datoteke, ki tja ne sodijo.

Imam še nekaj komentarjev glede ER diagrama:

  • Trenutno imate eniško relacijo igra. Na ER diagramih eniških relacij navadno ne rišemo, saj jih lahko navadno predstavimo z atributi. Sklepam pa, da tukaj ni bilo tako mišljeno - verjetno je bila mišljena povezava med igralcem in klubom?
  • Relacije se dogovarja, najde in se zmeni se najbrž nanašajo tudi na igralca oziroma potencialen prestop, tako da bo treba dodati še ustrezne povezave. Morda lahko ločite med potencialnimi in dejansko izvedenimi prestopi (najbolje kar z dodatnim atributom pri entiteti Prestop).
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

No branches or pull requests

1 participant