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

Login frontend #25

Merged
merged 5 commits into from
Mar 2, 2024
Merged

Login frontend #25

merged 5 commits into from
Mar 2, 2024

Conversation

masinnae
Copy link
Contributor

  • (minimalistische) login pagina is klaar
  • knop doet nog niks
  • wanneer auth backend klaar is, verder kijken om login in frontend te laten werken

frontend/src/assets/login.css Outdated Show resolved Hide resolved
frontend/src/assets/universiteit-gent-logo-white.png Outdated Show resolved Hide resolved
frontend/src/assets/logo_white_ transparant.png Outdated Show resolved Hide resolved
Copy link
Contributor

@reyniersbram reyniersbram left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ziet er al goed uit, maar de codestijl is niet echt volgens de Composition API van Vue.js. Ik wil hier gerust eens voor bellen ofzo om wat dingen uit te leggen als dat nodig is.
Om het aan te passen kan je zeker eens kijken naar de skeleton code, die is zeer duidelijk en toont eigenlijk wel exact hoe het zou moeten.

frontend/src/main.ts Outdated Show resolved Hide resolved
<script>
import '../assets/login.css'

export default {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hier maak je gebruik van de Options API van Vue.js, wij gebruiken echter de Composition API. Dit is wel een groot verschil op vele vlakken, dus houdt hier zeker rekening mee als je dingen op zoekt.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dit wordt automatisch gegenereerd wanneer je een nieuwe .vue file aanmaakt, misschien kan dit ergens in de settings wel aangepast worden?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Welke editor gebruik je? Misschien dat dat in de instellingen wel ergens gekozen kan worden.

@masinnae
Copy link
Contributor Author

masinnae commented Feb 26, 2024

  • volgt nu het Single-File principe
  • UGent logo tijdelijk weggehaald tot een beslissing over het thema genomen is
  • automatisch gegenereerde Options API code weggehaald
  • Router naar LoginView.vue ipv App.vue overschrijven

commit 3008695

frontend/src/App.vue Outdated Show resolved Hide resolved
In App.vue heb ik wat van de skeleton code teruggeplaatst zodat dit niet enkel die login knop is. Kan misschien ook handig zijn om hier later dan nog op terug te vallen om te zien hoe dingen precies gedaan worden.
@reyniersbram reyniersbram added the frontend This is related to the frontend label Mar 2, 2024
@masinnae masinnae marked this pull request as ready for review March 2, 2024 14:04
@reyniersbram reyniersbram merged commit 5617f9f into main Mar 2, 2024
1 check passed
@masinnae masinnae deleted the login_frontend branch March 10, 2024 18:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
frontend This is related to the frontend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants