Katselmointi

Sivuston käytettävyys

Mitä tein

15.8.2019 klo ~11:50
Rekisteröidyin ja kirjauduin sisään
- Lähetin chat viestin

Kokemus

Olisi kiva jos rekisteröinnistä ohjattaisiin suoraan sisälle.
Siisti about sivu, ehkä sivun voisi jopa laittaa rekisteröinnin ulkopuolelle

Koodi

Koodista näkee että ratkaisuihin on käytetty aikaa ja vaivaa 😍
Guard clausen käyttö voi auttaa luettavuutta, lue https://refactoring.com/catalog/replaceNestedConditionalWithGuardClauses.html ja https://reactjs.org/docs/conditional-rendering.html :

https://github.com/stutommi/kaste_2.0-frontend/blob/e95ee887cd716a7b6723beebd77031d8830b7271/src/App.js#L38-L86 vs

if (!token) return <LoginView setToken={setToken} />

return (
  <ResponsiveLayout
    page={page}
    setPage={setPage}
    logOut={logOut}
    actions={actions}
    token={token}
    sensorService={sensorService}
  >

    <SensorView
      token={token}
      sensorData={sensorData}
      sensorError={sensorError}
      show={page === 'Sensors'}
    />

    <ChatView
      show={page === 'Chat'}
    />

    <AboutView
      show={page === 'About'}
    />

    <SettingsView
      show={page === 'Settings'}
      sensorsConnected={sensorsConnected}
      token={token}
      setToken={setToken}
      actions={actions}
      sensorService={sensorService}
    />

    <VideoView
      show={page === 'Video'}
      token={token}
      actions={actions}
    />
  </ResponsiveLayout>
)

Kuten edelliseen on myös https://github.com/stutommi/kaste_2.0-frontend/blob/master/src/components/VideoView.js conditional rendering voisi muuttaa muotoon const [pending, setPending] = useState(true) ja vastaavasti onLoad={() => setPending(false)}
Kehitysidea: ota käyttöön typescript, kun projektina oli muutenkin kirjoittaa uudestaan niin opit uutta
Voit miettiä muita ratkaisuja miten saisi frontin paketoitua lennosta backendin kanssa. Suosittelen Dockeria.

Kokonaisuus

Kekselijäs idea, harmi etten päässyt näkemään / demoamaan paljoa toiminnallisuudesta mutta ei se mitään.
Hieno projekti! Toivottavasti keksitte jatkokehitettävää

🌱/ 🏡

Kiitos Jami paljon katselmoinnista, tosi hyviä pointteja! Korjasin jo tuon videoview- ja app-komponentin, näyttää kieltämättä heti paremmalta.

Lopetin juuri tekemäsi kurssin (kiitos vielä siitä, oli erittäin opettavainen), olisi mielenkiintoista kyllä laittaa koko homma pyörimään containereihin!

Ikävä ettet nähnyt toiminnallisuutta käytännössä, lähetin kyllä Matille linkkinä sensorendpointin, jonka kautta kaiken toiminnallisuuden olisi saanut näkymään, se ei varmaan tullut sinulle asti.

Itsellä tulikin tuon projektin jälkeen halu ottaa TypeScriptiä haltuun (ja lisätä Reduxiakin portfolioon), niin tein sitä varten ihan oman projektinsa (https://github.com/stutommi/familyLog-frontend). Siinä sai samalla myös korjattua kasteen puutteen kunnon rekisteröitymisessä sähköpostin kanssa ja muita, joitain muita asioita, joita jäi kasteessa kokeilematta.

Jos kiinnostaa/jäi kaivelemaan niin voin lähettää tunnukset käyttäjätiliin, jossa on kaikki toiminnallisuus mukana. Koska niissä voi oikeasti kastella eläviä kasveja niin en uskalla niitä yleiseen jakoon pistää, ettei tule vesivahinkoa.. :D

Kiitos vielä kerran katselmuksesta!

stutommi / kaste_2.0-frontend