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 mining and stacking functionality #5

Closed
wants to merge 49 commits into from

Conversation

whoabuddy
Copy link
Collaborator

No description provided.

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Sep 21, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: f124c9b
Status: ✅  Deploy successful!
Preview URL: https://728bab14.citycoins2.pages.dev
Branch Preview URL: https://feat-mining-stacking-1.citycoins2.pages.dev

View logs

Not working just yet, need to review against hook pattern on other loadables, and previous UI. Currently complaining about the write function in the atom family. Needs to be one atom per component for it's content, may require or at least justify splitting code here.
Something is happening with the fetch: it runs constantly in the bg instead of once. Nots ure if it's rerenders or something in the structure. Next step is to simplify hooks: one hook, one purpose.
Also add useMemo() to try and get stable atom family values for each query. Still looping somewhere, now for User IDs.
This fixes the fetch loop issue and creats a pattern to accept default values that can be set later.
Seeing way too many re-renders at this version but wanted to save as a checkpoint.
After a few iterations it seems like having the local storage and fetch components separate, then deriving them together is going to be the best approach.

Working now but not persisting in local storage, need more logging to troubleshoot.
@whoabuddy
Copy link
Collaborator Author

Closing this as an artifact, new direction taken with most components but has some good ideas for querying/displaying user information.

@whoabuddy whoabuddy closed this May 20, 2024
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.

1 participant