Mijn-9682-Feature - blokkeren stadspas with proper swr #1708
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
RoanPaulus
merged commit 7d5cda7
into
MIJN-9682-Feature/blokkeren-stadspas-with-swr-cache
3 checks passed
RoanPaulus
deleted the
MIJN-9682-Feature/blokkeren-stadspas-with-swr-cache-fix-swr
branch
timvanoostrom
added a commit
that referenced
this pull request
Jan 17, 2025
* added mock route * Added block URL to data * Added conditional button to block stadspassen * Moved logic * fixed tests * TODO comments * remove imports * Refactored out function for use in block pass * Made modal width resemble figma design * semi working version * small refactor * shows geblokkeerd after pas * Fix all stadspassen having the same ID * fix test * Add error alert * Put list of stadspas owners into a table for the 'actief' status * stopped updating whole appstate. Using atom instead * replaced with SWR functionality * Moved hook to hooks file * added mock route * Added block URL to data * Added conditional button to block stadspassen * Moved logic * fixed tests * TODO comments * remove imports * Refactored out function for use in block pass * Made modal width resemble figma design * semi working version * small refactor * shows geblokkeerd after pas * Fix all stadspassen having the same ID * fix test * Add error alert * Put list of stadspas owners into a table for the 'actief' status * stopped updating whole appstate. Using atom instead * replaced with SWR functionality * Moved hook to hooks file * use cache instead of atom, but does not rerender when updating cache * Corrected a type that got changed after a rebase with main * Moved routers next to eachother to make it more readable when you want to see which routers are in use * Refactor some types * Added route to the private stadspas handlers * Added route to openapi docs * Added tests for backend * updated snapshot * Added to test, but broken. Blocked pass won't show up yett * Mijn-9682-Feature - blokkeren stadspas with proper swr (#1708) * Fix types * Fix implementation * Refine types * Refine type * Juggle around references, change name * Change mock server responses (add dynamic props) * Change name * Change http verb * Rename hook * Enable delay * Change markup * FIx type * Add deprecation (boyscout) * Rename * Styling * ADd hack * Fix test * Fix gpass tests * Fix * Addes tests, made helper to create components * Add test for blocked pas --------- Co-authored-by: Tim van Oostrom <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.