reyesoft / ngx-jsonapi

JSON API client library for Angular 5+ 👌 :: Production Ready 🚀
https://ngx-jsonapi.reyesoft.com/
MIT License
101 stars 52 forks source link

Raise coverage #119

Closed maxi7587 closed 6 years ago

maxi7587 commented 6 years ago

Estuve un rato con el http service y me di cuenta de por qué no funciona el pipe share()... al hacer el request en el mismo método, justo antes de agregar el pipe, siempre se trata de un observable nuevo. Dejé comentado el test que está fallando y también un test de ejemplo con un pipe share() funcionando y una muetra de por qué no funciona. Un avez uqe lo veas avisame y lo borro para poder mergear.

pablorsk commented 6 years ago

¿Y si hacemos esto? Al menos temporalmente

https://github.com/reyesoft/ngx-jsonapi/blob/cd063ec3058898a0604c268f3188694495c0122a/src/sources/http.service.ts#L22-L25

por

  let obs;
  if (method === 'get') {
      obs = this.http
           .request<IDocumentData>(method, this.rsJsonapiConfig.url + path, req)
           .pipe(share());
  } else {
      obs = this.http
           .request<IDocumentData>(method, this.rsJsonapiConfig.url + path, req);
  }
maxi7587 commented 6 years ago

Dejo el comentario poruqe nos vamos a olvidar... XD... es lo que hablamos en la daily lo acabo de ver. Habría que crear la tarea para arreglarlo... ¿lo agregamos como issue? Si querés que la arme avisame :+1: