-
Notifications
You must be signed in to change notification settings - Fork 66
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
Create Vets::Collection #19895
base: master
Are you sure you want to change the base?
Create Vets::Collection #19895
Conversation
* re-add collection * linting
* add pagination class to collection * add new tests to collection and pagination * fix tests * linting * remove new requires that should be loaded * fix formatting * linting * fix will_paginate logic and add test
Generated by 🚫 Danger |
* add finder class and spec * complete finder class and spec * adjust naming * integrate finder with collection * linting * linting whitespace * fix collection specs
I'm putting this in draft until I can review with Eric |
This is parent branch that will be merged when other PRs are merged into itself
Summary
I will review & update metadata & errors between Vets Model and Collection in another PR
Related issue(s)
Testing done
Acceptance criteria