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 resources/program-studies page #508

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

Conversation

Woozl
Copy link
Collaborator

@Woozl Woozl commented May 31, 2024

image

This is unfortunately missing several features from the standalone app:

  • query caching
  • query parameters
  • full screen view

I can continue to work on these, but wanted to push this out as is for now. I began work on the useQuery hook to mirror the Tanstack API, but it was causing a lot of rerendering if there was an error, so I probably did something dumb. For now it's unused and the components simply fetch the necessary data directly.

@Woozl
Copy link
Collaborator Author

Woozl commented Jun 12, 2024

Query caching + query params are implemented. Full screen mode is still missing but I think it's okay without it for now--I increased the width of the container to help make the table more readable.

@Woozl Woozl requested a review from mbwatson June 12, 2024 14:09
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

1 participant