Skip to content

Conversation

Pikqi
Copy link

@Pikqi Pikqi commented Mar 24, 2023

No description provided.

@Pikqi Pikqi changed the base branch from main to codestantine March 24, 2023 16:02
@marsicdev
Copy link
Member

@Pikqi Hvala na rešenju, sve je tačno i kompletno međutim za merge je potrebno jos par ispravki:

  • Izvući router u novi fajl
  • Ukloniti console.log
  • srediti indentaciju i korišćenje ; da prati stil projekta

@Pikqi
Copy link
Author

Pikqi commented Mar 24, 2023

@Pikqi Hvala na rešenju, sve je tačno i kompletno međutim za merge je potrebno jos par ispravki:

  • Izvući router u novi fajl
  • Ukloniti console.log
  • srediti indentaciju i korišćenje ; da prati stil projekta

Hvala na feedbacku,
Radi konzistentosti stilizovanja koda u projektu, dodao sam prettierrc faijl koji mimikuje still koda.
Još neke od promena su:

  • Uklanjanje console log-a
  • Pomeranje router-a u odvojen fajl

@marsicdev
Copy link
Member

@Pikqi hvala, mozes mi poslati svoje podatke i najbitnije email na marko [at] hypetech.io kao bi dobio uputstva kako da preuzmes svoju kartu :)

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

Successfully merging this pull request may close these issues.

2 participants