Code review - prosta aplikacja [html + css + js + firebase]

Question

GH: https://github.com/Danie3L/list

Live: https://app.netlify.com/sites/tender-poincare-0082df/overview

 

Answer 1

• https://github.com/Danie3L/list/blob/master/firebase/firestore.js#L2
  Tego typu dane nie powinny być dostępne publicznie, ale brane np. z pliku .env, którego docelowa wersja nie jest commitowana, tylko wstawiana na serwer osobno.
• https://github.com/Danie3L/list/blob/master/index.html#L9
  Nie ma potrzeby używać atrybutu defer na module, ponieważ on domyślnie jest deferowany.
  Upewniłbym się też, czy podpięcie skryptu bezpośrednio po CSS-ie nie zblokuje parsera DOM (mimo użycia defer).
• https://github.com/Danie3L/list/blob/master/index.html#L26
  Użyj tutaj <input type="email">, dzięki czemu nie będziesz musiał samemu pisać RegExpa do jego walidacji.