0005/devise - Githubissues

BlkPingu / attendion

attendion is a free, open source calender app with emphasis on event management

0 stars 0 forks source link

0005/devise #36

Closed cyb3a closed 6 years ago

cyb3a commented 6 years ago

User creation and authentication with devise
update and display user information

BlkPingu commented 6 years ago

refactored app/views/welcome/index.html.erb to app/views/welcome/home.html.erb
added rule db/*.sqlite3 to .gitignore
added gemsbootstrap and bootstrap imports to application.scss
fixed navbar-fixed top partiallly covering login form with padding rule in application.scss

permission to merge?

viktorianer commented 6 years ago

Bitte eine Commit-Nachricht so wählen, dass aus Sicht des Commits klar wird, was er verändert. Zum Beispiel: "Change import of bootstrap" oder ähnlich...

Ein Commit kann man immer unabhängig betrachten. Ein Commit ist auch nie eine History, was passiert ist. Ein Commit ist jetzt, es verändert durch sich selbst etwas im Projekt. Deshalb auch nicht die Vergangenheitsform wählen.

Schauen Sie dazu die Git-Dokummentation, wenn nötig.

viktorianer commented 6 years ago

Stellen Sie bitte im Repo die Pull-Requests so ein, wie in Vorlesung gefordert. Gefordert ist, dass min. eine Zustimmung für Merge notwendig ist. Jetzt ist keine Zustimmung nötig.

Hier kann ein Merge erfolgen.

BlkPingu commented 6 years ago

Danke für die Kritik. Habe hier gute Informationen gefunden wie ich das in Zukunft besser machen kann.

~~Inhaltlich jetzt soweit alles in Ordnung dass wir mergen dürfen?~~ Edit: Danke! Werden wir in Zukunft.

viktorianer commented 6 years ago

Schöne Übersicht und deckt sich soweit mit der Vorlesung. Offiziell Dokumentation wäre mir aber lieber, die ich in meinen Übungen vorgestellt habe.