pekoe09/eqma

Eqma - equipment rental business management

eqma's Issues

Koodikatselmointi

Sivuston käytettävyys

• Monella sivulla on klikattavia elementtejä jotka ei kuitenkaan muuta hiiren kursoria, olisi hyvä että klikattavien elementtien kohdalla kursori muuttuu pointteriksi
  • Se onnistuu css:llä näin: cursor: pointer;
• etusivulla tuotteilla on kivat kuvat, mutta tuotesivulla kuvat puuttuu, olisi kiva nähdä kuvat myös siellä?
• logout napin painaminen ei tee mitään, sivun reloadin jälkeen napin painaminen kirjasi ulos

Frontend

• App.js on wrapattu connect funktioon, mutta sillä komponentilla ei ole lainkaan propseja käytössä

  • itseasiassa koko komponentin voisi muuttaa funktiokomponentiksi

• projektissa on reduxia käytetty hienosti

• kurssilla opittuja asioita sovelletaan myös hienosti

• momentjs:ää käytetty! Nice!

• customers/customerCreate.js tiedostossa on tämmöinen rivi

  const customer = {
    lastName: this.state.lastName,
    firstNames: this.state.firstNames,
    company: this.state.company,
    email: this.state.email,
    phone: this.state.phone,
    billingAddress: {
      street1: this.state.street1,
      street2: this.state.street2,
      zip: this.state.zip,
      city: this.state.city,
      country: this.state.country
    },
    isInvoicable: this.state.isInvoicable
  }
  

  Tämän voisi refaktoroida muotoon

  const {street1, street2, zip, city, country, ...values} = this.state
  const customer = {
    ...values,
    billingAddress: {
      street1,
      street2,
      zip,
      city,
      country
    }
  }
  

• equipment/equipmentCreate.js tiedostossa on tämmöisiä rivejä:

  <Form.TextArea width={12} rows={5} label='Description' name='description'
    value={this.state.description} onChange={this.handleChange} />
  

  Nämä voisi muotoilla luettavampaan muotoon

  <Form.TextArea
    width={12}
    rows={5}
    label='Description'
    name='description'
    value={this.state.description}
    onChange={this.handleChange}
  />
  

• suosittelen .eslintrc.js tiedostoon airbnb:n eslint konffin lisäämistä, sillä saa kivasti kiinni kaikenlaisia probleemia :)

Backend

• routeissa hienosti käytetty middlewareja!
• paljon testejä, hyvä homma!
• mongoa käytetty hyvin
• koodi on hyvää, controllers/images.js tiedostossa pari tyhjää routea
• customerMessages.js:n olisi voinut lisätä validointimiddlewarea?

Kaiken kaikkiaan tosi hyvin tehty projekti! Tästä on hyvä jatkaa :)