Skip to content
This repository has been archived by the owner on Feb 1, 2024. It is now read-only.

Expand ExtendedField.FIELD_CHOICES #1595

Merged
merged 1 commit into from Jan 20, 2022
Merged

Conversation

TaiWilkin
Copy link
Contributor

@TaiWilkin TaiWilkin commented Jan 19, 2022

Overview

Creates a data migration to add additional ExtendedField.FIELD_CHOICES,
specifically facility_type, processing_type, product_type, and
parent_company.

Connects #1581

Demo

Screen Shot 2022-01-19 at 12 09 55 PM

Notes

I temporarily rebased this on the expanded facility details PR, and was able to manually add extended fields of the new types and view them in the facility details.

Testing Instructions

  • Run ./scripts/manage migrate
  • Run ./scripts/server
  • Navigate to the extended fields admin and update or create an ExtendedField.
  • The list of field types should include facility_type, processing_type, product_type, and parent_company.

Checklist

  • fixup! commits have been squashed
  • CI passes after rebase
  • CHANGELOG.md updated with summary of features or fixes, following Keep a Changelog guidelines

Copy link
Contributor

@jwalgran jwalgran left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@jwalgran jwalgran assigned TaiWilkin and unassigned jwalgran Jan 19, 2022
Creates a data migration to add additional ExtendedField.FIELD_CHOICES,
specifically facility_type, processing_type, product_type, and
parent_company.
@TaiWilkin TaiWilkin force-pushed the tw/add-extended-field-choices branch from 7e69c21 to 439fd03 Compare January 20, 2022 13:32
@TaiWilkin
Copy link
Contributor Author

Thanks for reviewing!

@TaiWilkin TaiWilkin merged commit f82b26a into develop Jan 20, 2022
@TaiWilkin TaiWilkin deleted the tw/add-extended-field-choices branch January 20, 2022 13:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants