IEEE-NITK / corpus

A Django-based web application for managing all things IEEE NITK!
https://ieee.nitk.ac.in
2 stars 7 forks source link

Added Model for SIGs #60

Closed imApoorva36 closed 11 months ago

imApoorva36 commented 11 months ago

Description

Added model for SIGs

Fixes # (issue)

Dependencies

None new

Type of Change

What types of changes does your code introduce? Put an x in the boxes that apply

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Please also list any relevant details for your test configuration. (for bug fix / breaking change) Put an x in the boxes that apply


Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

anirudhprabhakaran3 commented 11 months ago

Move the model to the pages app.

nishant-nayak commented 11 months ago

For making any DB related changes, I would highly recommend you check out the ER Diagrams in the wiki. I've documented the fields required, the type of the field and any DB constraints that we should apply.

For this case, the SIG model should go in the config app, alongside the Society model.

anirudhprabhakaran3 commented 11 months ago

Aah yes, my bad. I didn't check the ER diagram before commenting that 😓

anirudhprabhakaran3 commented 11 months ago

Bruh wait, we need to figure out how to restore those migrations you've deleted.