Skip to content
Snippets Groups Projects

Make organizer default address required

Merged Make organizer default address required
require-organizer-address into master
1 unresolved thread
Merged Jari Rentsch requested to merge require-organizer-address into master
1 unresolved thread

As address has a foreignkey to organizer, we need to keep the default_address nullable. However we require it in the forms.

Additionally I improved the address tests

Review !291 (merged) first

Edited by Jari Rentsch

Merge request reports

Pipeline #2347 passed

Stage:

    Pipeline passed for 369209fc on require-organizer-address

    Test summary results are loading
    Full report

    Merged by Jari RentschJari Rentsch 6 months ago (Dec 10, 2024 9:31am UTC)

    Loading

    Pipeline #2348 passed

    Stage:
      Stage:
        Stage:

          Pipeline passed for 369209fc on master

          Activity

          Filter activity
          • Approvals
          • Assignees & reviewers
          • Comments (from bots)
          • Comments (from users)
          • Commits & branches
          • Edits
          • Labels
          • Lock status
          • Mentions
          • Merge request status
          • Tracking
          championship/models.py
          187 192 default_address = models.ForeignKey(
          188 193 Address,
          189 194 on_delete=models.SET_NULL,
        • Antoine Albertelli added 2 commits

          added 2 commits

          • ec2cfe04 - Make organizer default address required
          • c0890444 - Use the DB to protect Address Foreign Keys

          Compare with previous version

        • Antoine Albertelli added 1 commit

          added 1 commit

          • 369209fc - Use the DB to protect Address Foreign Keys

          Compare with previous version

        • Jari Rentsch merged

          merged

          • Please register or sign in to reply