s0lvang / Karantenehjelpen

A web-application for helping people in quarantine to get groceries
https://karantenehjelpen.no
27 stars 6 forks source link

Flytt firebasefunksjoner til service-fil #171

Closed FredrikAugust closed 4 years ago

FredrikAugust commented 4 years ago

Fjerna også axios siden det bare ble brukt ett sted, og konvertering til fetch var veldig greit.

Og slett bruker (firebase.Auth) når du sletter brukeren, ikke bare brukerdata.

FredrikAugust commented 4 years ago

Patternet jeg kjørte hadde grunnlag i dette;

De fleste funksjoner kjørte en firebase-query/update, for så å oppdatere lokal state / endre path, så fant det logisk at alle firebase-wrapperne hadde formatet (e.g.)

const queryExample = async (userId: string, requestId: string, callback: (request: IRequest) -> void) => {
  try {
    const doc = await fb.doc(userId).collection('requests').doc(requestId);

    callback(doc);
  } catch (err) {
    consonle.error(`Error oppsto ved henting av ordre: ${err}`);
  }
}
methods: {
  example() {
    queryExample("bruker1", "request1", doc => {
      this.doc = doc;
      this.$router.push('/joikaboller');
    });
  }
}

Dermed slipper du å styre med at du har feil this i wrapper-funksjonen, og lar heller caller styre med det.

oleast commented 4 years ago

Tenker at hvis vi kan se på code-splitting delen in en separat PR uansett, så like greit å kjøre dette gjennom før man begynner å se på endringer som det? :small_airplane:

FredrikAugust commented 4 years ago

LGTM <3

Kanskje greit å teste rundt på dev en del med denne ettersom den har endringer på veldig viktig logikk?

Yes, tror det er lurt. Får du merga og devsatt hvis det bra ut?

FredrikAugust commented 4 years ago

@kharann Har du mulighet til å devsette denne?

kharann commented 4 years ago

Æg devsetter