Skip to content
This repository has been archived by the owner on Oct 28, 2020. It is now read-only.

Avoid double subscriptions #78

Closed
ZimGil opened this issue Nov 10, 2018 · 0 comments · Fixed by #98
Closed

Avoid double subscriptions #78

ZimGil opened this issue Nov 10, 2018 · 0 comments · Fixed by #98
Assignees
@ZimGil
Labels
bug Something isn't working client
Milestone
Beta Release

Comments

@ZimGil
Copy link
Owner

ZimGil commented Nov 10, 2018

We should avoid using "Double Subscriptions" (subscribe within another subscribe).
We took some steps to avoid it at #77 in src/app/app.component.ts
@ZimGil ZimGil added bug Something isn't working client labels Nov 10, 2018
@ZimGil ZimGil added this to the Beta Release milestone Nov 10, 2018
@ZimGil ZimGil self-assigned this Nov 10, 2018
@ZimGil ZimGil mentioned this issue Nov 23, 2018
Merged
ZimGil added a commit that referenced this issue Nov 23, 2018
@ZimGil
Merge pull request #98 from ZimGil/#78-avoid-double-subscriptions
16d2952 
Removed double subscriptions and unsubscriptions
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@ZimGil ZimGil

Labels
bug Something isn't working client
Projects
None yet
Milestone
Beta Release
Development

Successfully merging a pull request may close this issue.

Removed double subscriptions and unsubscriptions ZimGil/ngWiz
1 participant
@ZimGil