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

feat: add rxjs operators poll and whenDocumentVisible #404

Merged

Conversation

FabienDehopre
Copy link
Contributor

@FabienDehopre FabienDehopre commented Jun 3, 2024

Add two RxJs operators that works well together but still can be used independently.

poll can be used to poll an API at a fixed interval with an optional initial delay.
whenDocumentVisible is the RxJs counterpart of the injectDocumentVisibility and can be used to pause and resume a RxJs stream based on the document visibility state.

Related to #405

@FabienDehopre FabienDehopre marked this pull request as ready for review June 3, 2024 09:55
@eneajaho
Copy link
Collaborator

eneajaho commented Jun 6, 2024

PR not found

@eneajaho eneajaho requested a review from nartc June 6, 2024 13:23
@FabienDehopre
Copy link
Contributor Author

What do you mean?

@nartc
Copy link
Collaborator

nartc commented Jun 10, 2024

@all-contributors please add @FabienDehopre for code

@nartc
Copy link
Collaborator

nartc commented Jun 11, 2024

@all-contributors please add @FabienDehopre for code

@nartc
Copy link
Collaborator

nartc commented Jun 11, 2024

@FabienDehopre looks like the test failed. Can you take a look?

@nartc
Copy link
Collaborator

nartc commented Jun 11, 2024

@all-contributors please add @FabienDehopre for code

(third time the charm?)

Copy link
Contributor

@nartc

I've put up a pull request to add @FabienDehopre! 🎉

@nartc
Copy link
Collaborator

nartc commented Jun 11, 2024

@all-contributors please add @FabienDehopre for code

(fourth time 🥲)

Copy link
Contributor

@nartc

@FabienDehopre already contributed before to code

@FabienDehopre
Copy link
Contributor Author

I've fixed the failing tests.

@nartc nartc merged commit 7ffc766 into ngxtension:main Jun 16, 2024
1 check passed
@FabienDehopre FabienDehopre deleted the feature/poll-when-document-visible branch June 17, 2024 07:38
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

Successfully merging this pull request may close these issues.

None yet

4 participants