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.

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

Update migration idempotence, add org email field, extra project fields #2072

Merged
merged 3 commits into from
Jan 10, 2025

Conversation

spwoodcock
Copy link
Member

@spwoodcock spwoodcock commented Jan 9, 2025

What type of PR is this? (check all applicable)

  • πŸ• Feature
  • πŸ› Bug Fix
  • πŸ“ Documentation
  • πŸ§‘β€πŸ’» Refactor
  • βœ… Test
  • πŸ€– Build or CI
  • ❓ Other (please specify)

Related Issue

#2066
#1979
#1985

Describe this PR

  • Tweaks the idempotence of some previous migrations.
  • Added extra fields:
    • organisation.associated_email
    • project.geo_restrict_distance_meters (default 50) and geo_restrict_force_error (default false)
    • project.new_geom_type with enum options 'POINT', 'POLYGON', 'LINESTRING'
  • Also added these fields to the backend API models: the are returned in responses + allowed during creation POST.

Review Guide

Notes for the reviewer. How to test this change?

Checklist before requesting a review

[optional] What gif best describes this PR or how it makes you feel?

@spwoodcock spwoodcock requested a review from Sujanadh January 9, 2025 16:08
@github-actions github-actions bot added backend Related to backend code migration Contains a DB migration labels Jan 9, 2025
@spwoodcock
Copy link
Member Author

@Sujanadh please merge if you are happy with it πŸ™

@Sujanadh
Copy link
Collaborator

looks good πŸ‘

@Sujanadh Sujanadh merged commit 056c3b5 into development Jan 10, 2025
6 checks passed
@Sujanadh Sujanadh deleted the build/extra-db-fields branch January 10, 2025 04:44
@Sujanadh Sujanadh mentioned this pull request Jan 10, 2025
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Related to backend code migration Contains a DB migration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants