fix(connectObs): unsubscribe from observables when unmounting

[matthieuprat]

No description provided.

[julien-meichelbeck]

A monster. 👾

[codecov]

Codecov Report

Merging #92 into master will increase coverage by 0.04%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #92      +/-   ##
==========================================
+ Coverage   96.52%   96.57%   +0.04%     
==========================================
  Files          63       63              
  Lines         461      467       +6     
==========================================
+ Hits          445      451       +6     
  Misses         16       16

Impacted Files	Coverage Δ
src/connectObs.js	92.85% <100%> (+1.19%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6d3650a...57dc917. Read the comment docs.

[matthieuprat]

I tested this changeset against the entire @doctolib code base and it's all green on CI.

[oliviertassinari]

Ça leak 🌊. Est-ce que c'est ça que j'avais sur l'agenda mobile docteur ?

[julien-meichelbeck]

C'est en rajoutant une feature sur l'agenda mobile docteur que j'ai remarqué ça, donc très probablement :)

à chaque swipe vers le prochain jour, on gardait tous les observables sur les jours précédents :'(

[oliviertassinari]

Well done!

[gregberge]

Incroyable que l'on ne l'ai pas vu avant 😱

[julien-meichelbeck]

ouais ça faisait une requête xhr de plus à chaque swipe ...