Skip to content
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.

Sign up for GitHub

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

Jump to bottom

390 Remove enums from Zoning District Class #396

Merged
merged 1 commit into from
Dec 31, 2024
Merged

390 Remove enums from Zoning District Class #396

merged 1 commit into from
Dec 31, 2024

Conversation

horatiorosa
Copy link
Contributor

@horatiorosa horatiorosa commented Dec 11, 2024 •
edited
Loading

Description

  • Removed the categoryEnum from zoning-district-class
  • Changed the category field from a categoryEnum to a text
  • Added a check constraint to the category field

Tickets

Closes #390

@horatiorosa horatiorosa self-assigned this Dec 11, 2024
@horatiorosa horatiorosa force-pushed the Issue/390-remove-enums-from-zoning-district-class branch from e31c8a5 to 9ef767a Compare December 11, 2024 20:10
@horatiorosa horatiorosa marked this pull request as ready for review December 12, 2024 14:25
@horatiorosa horatiorosa changed the title drop enums, update types from enum to text, add constraints 390 drop enums, update types from enum to text, add constraints Dec 18, 2024
@horatiorosa horatiorosa changed the title 390 drop enums, update types from enum to text, add constraints 390 Remove enums from Zoning District Class Dec 18, 2024
@horatiorosa horatiorosa force-pushed the Issue/390-remove-enums-from-zoning-district-class branch 2 times, most recently from 1b78769 to 0358dc1 Compare December 26, 2024 15:26
@horatiorosa horatiorosa force-pushed the Issue/390-remove-enums-from-zoning-district-class branch from cad2eef to 77a70a6 Compare December 31, 2024 15:01
@horatiorosa horatiorosa merged commit 2386fa3 into main Dec 31, 2024
3 checks passed
@horatiorosa horatiorosa deleted the Issue/390-remove-enums-from-zoning-district-class branch December 31, 2024 18:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@TangoYankee TangoYankee TangoYankee approved these changes

Assignees

@horatiorosa horatiorosa

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Refactor zoning district class category from enum to check constraint
2 participants
@horatiorosa @TangoYankee