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

Add read untracked to signals #1593

Merged
merged 5 commits into from
Dec 7, 2023

Conversation

ealmloff
Copy link
Member

Adds a read untracked method to Signals. This method is like read but it doesn't subscribe to the current context. This can be useful for signals you do not want to track inside of a dioxus_signals::effect

@ealmloff ealmloff added enhancement New feature or request signals Related to the signals crate labels Oct 30, 2023
@ealmloff ealmloff added this to the 0.5.0: Signals milestone Nov 21, 2023
@jkelleyrtp
Copy link
Member

Can we rename this to peek and then mention it in the docs for regular reads?

@jkelleyrtp jkelleyrtp self-requested a review December 7, 2023 05:18
@jkelleyrtp
Copy link
Member

Just FYI peak and peek are two different spellings. We want peek (peak is like for mountains)

@jkelleyrtp jkelleyrtp enabled auto-merge December 7, 2023 19:12
@jkelleyrtp jkelleyrtp merged commit 028ffad into DioxusLabs:master Dec 7, 2023
@ealmloff ealmloff deleted the signal-read-untracked branch December 7, 2023 19:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request signals Related to the signals crate
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants