Skip to content

issue 39 create Registration models #42

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

Merged
merged 6 commits into from
Jul 10, 2025
Merged

Conversation

Viloetsisi
Copy link
Contributor

Change Summary

This PR adds the new Registration model to the event_registration app:

  • UUID primary key
  • ForeignKey to User (1-to-many)
  • status enum with choices draft|submitted|paid
  • Process fields (registration_category, age_group, tshirt_size, etc.)
  • Progress & timestamp fields (current_step, last_saved_at, submitted_at, created_at, updated_at)

Change Form

  • The pull request title includes the issue number (issue-39-Create_model_for_registration)
  • The change has been smoke-tested in the Django shell (created UserRegistration, verified forward/reverse relations)
  • The change has tests
    Reason: No unit tests added in this PR; will cover testing in a follow-up.
  • The change has documentation
    Reason: Model fields are self-explanatory; full docs to accompany API/serializer work.

Other Information

  • After merging, run:
    python manage.py makemigrations event_registration
    python manage.py migrate

@Viloetsisi Viloetsisi changed the title Add Registration (and RegistrantDetails) models Add Registration models Jul 5, 2025
@Viloetsisi Viloetsisi changed the title Add Registration models issue 39 create Registration models Jul 5, 2025
@Viloetsisi Viloetsisi requested a review from MahitGtg July 7, 2025 06:53
Copy link
Collaborator

@MahitGtg MahitGtg left a comment

Choose a reason for hiding this comment

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

lgtm

@Viloetsisi Viloetsisi merged commit bcf7b8d into main Jul 10, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants