Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RXJS subscriptions not always cleared #121

Open
2 of 8 tasks
waratah opened this issue Sep 12, 2022 · 1 comment
Open
2 of 8 tasks

RXJS subscriptions not always cleared #121

waratah opened this issue Sep 12, 2022 · 1 comment

Comments

@waratah
Copy link
Contributor

waratah commented Sep 12, 2022

I'm submitting a

  • Regression (a behavior that used to work and stopped working in a new release)
  • Bug report
  • Enhancement
  • Feature request
  • Documentation issue or request

Description

  • Bug: Rxjs subscription not removed in all cases
  • Feature: Description of the feature. Cause and its Motivation_

RXJS will keep sending updates until it is completed. I use this over a list that constantly pushes updates and the autocomplete itself keeps coming and going. This leaves a number of these subscriptions open consuming memory and processing useless code on the update of the initial list.

I have implemented a work around by using take(1) on my subscription however this bug is not obvious. I only found it as a result of looking at the code. There is also a slight possibility that the wrong list could be displayed, I never saw it happen however.

This also introduces a lookup when the list is cleared. Our testers considered it an error that I entered part of the values then the list just disappeared entirely. This is a breaking change as the results are different from the previous version.

Version Information

ionic4-auto-complete: 2.9.9
@angular/cli: 10.0.2

Browser

  • Chrome - 105.0.5195.102

NOTE: Replace x.x.x with your version information

@waratah
Copy link
Contributor Author

waratah commented Sep 12, 2022

#120

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant