-
Notifications
You must be signed in to change notification settings - Fork 24
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
Migrate unit summary amiPercentage to int #1797
Migrate unit summary amiPercentage to int #1797
Conversation
* add migration for ami percentage column * Fix code style issues with Prettier * fix validation for amiPercentage Co-authored-by: Lint Action <lint-action@samuelmeuli.com>
❌ Deploy Preview for dev-bloom failed. 🔨 Explore the source changes: d0a8f9e 🔍 Inspect the deploy log: https://app.netlify.com/sites/dev-bloom/deploys/613f695a917cbe0008474e52 |
✔️ Deploy Preview for dev-partners-bloom ready! 🔨 Explore the source changes: d0a8f9e 🔍 Inspect the deploy log: https://app.netlify.com/sites/dev-partners-bloom/deploys/613f695a4e5f430007ea912c 😎 Browse the preview: https://deploy-preview-1797--dev-partners-bloom.netlify.app |
✔️ Deploy Preview for dev-storybook-bloom ready! 🔨 Explore the source changes: d0a8f9e 🔍 Inspect the deploy log: https://app.netlify.com/sites/dev-storybook-bloom/deploys/613f695adff72100073c742b 😎 Browse the preview: https://deploy-preview-1797--dev-storybook-bloom.netlify.app |
@seanmalbert and @willrlin for visibility |
Also looks like linters are still failing on forks. |
And the failing test is a rate-limiting issue. |
@pbn4 , can you please review. |
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.
Thank you, @avaleske, looks good!
* Migrate unitSummary amiPercentage column to int (#534) * add migration for ami percentage column * Fix code style issues with Prettier * fix validation for amiPercentage Co-authored-by: Lint Action <lint-action@samuelmeuli.com> * update changelog Co-authored-by: Lint Action <lint-action@samuelmeuli.com> Co-authored-by: seanmalbert <smabert@gmail.com>
Pull Request Template
Issue
Migrate unit summary AMI Percentage field to integer, to support filtering
Addresses CityOfDetroit#478
Description
Migrate the amiPercentage column on the unit summary table to an integer, so we can filter based on it.
Type of change
How Can This Be Tested/Reviewed?
Checklist: