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

Refactor Yewdux state management #19

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

intendednull
Copy link

@intendednull intendednull commented Dec 20, 2023

Greetings! I recently stumbled onto this project from YouTube. Loved following your Yew/Rust journey!

There were a few questions about overall state management and structure towards the end. You decided to move away from shared state (which is a perfectly reasonable approach), however I'd like to propose an alternative. These changes demonstrate how you might continue to use Yewdux for all your state management needs.

I do not expect this to be merged. My primary goal was to provide a working solution, which will hopefully be a useful reference. By no means is it better or more "correct" than the current version, just another take on an interesting problem.

I recorded my thought process here, hopefully it's interesting and informative :)

Adds RepoFilter, which is much more flexible.
Prevents the extra request that was happening at the end of loading an
organization. Was initially removed for simplicity, but re-added because
we waste no cycles here :)
@NicMcPhee
Copy link
Owner

Thank you so much for the kind words about the stream, and for taking the time to contribute like this! I'm swamped with family things at the moment, but I'll try to come back to this in the next week or two and look this over.

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.

2 participants