-
Notifications
You must be signed in to change notification settings - Fork 40
feat: paginated msq list endpoint #5797
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
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for bloom-public-seeds ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for bloom-lakeview ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for bloom-flagly ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for bloom-exygy-dev ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for bloom-angelopolis canceled.
|
✅ Deploy Preview for partners-bloom-dev ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
jaredcwhite
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good @mcgarrye, just one issue leading to a failing test and type question.
|
|
||
| const getCardContent = () => { | ||
| if (!loading && data?.length === 0) return null | ||
| if (!loading && data?.meta.totalItems === 0) return null |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's a failing Cypress test because data?.meta could be undefined. Not sure what's going on with that.
| applicationSection?: MultiselectQuestionsApplicationSectionEnum | ||
| ) => { | ||
| data: SelectAndOrderSection[] | ||
| data: PaginatedMultiselectQuestion |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we even need the SelectAndOrderSection type anymore? Seems just to be an alias.
Description
For the new Preferences table with V2 MSQ, we need to have metadata for pagination on the list endpoint.
This is backwards compatible and does not require a feature flag. Frontend files are update to handle receiving the metadata.
How Can This Be Tested/Reviewed?
Login to the partners site
Go to the Settings page and make sure the Preferences table loads
Create/Edit a listing
Make sure page loads
Add a Preference or Program
Author Checklist:
yarn generate:clientand/or created a migration when requiredReview Process: