Skip to content

regenerate discipline migrations#175

Open
rayyanmridha wants to merge 4 commits intomainfrom
rm-173-regenerate-discipline-migrations
Open

regenerate discipline migrations#175
rayyanmridha wants to merge 4 commits intomainfrom
rm-173-regenerate-discipline-migrations

Conversation

@rayyanmridha
Copy link
Collaborator

@rayyanmridha rayyanmridha commented Feb 16, 2026

ℹ️ Issue

Closes #173

📝 Description

I regenerated the migrations for discipline.

✔️ Verification

I don't have a picture before \dt without discipline since it existed before but here it is after:
Screenshot 2026-02-17 at 8 56 04 PM

Test GET endpoint to get the whole table:
Screenshot 2026-02-17 at 9 13 18 PM

Test GET endpoint to get a discipline:
Screenshot 2026-02-17 at 9 01 24 PM

Test POST endpoint to add a discipline:
Screenshot 2026-02-17 at 9 02 27 PM

Test POST endpoint to add a discipline with an admin (adds admin to existing discipline):
Screenshot 2026-02-17 at 9 03 13 PM

Test DELETE endpoint to delete a discipline with an admin:
Screenshot 2026-02-17 at 9 04 08 PM

Test DELETE endpoint to delete a discipline:
Screenshot 2026-02-17 at 9 04 46 PM

Table description:
Screenshot 2026-02-17 at 8 59 26 PM

Current state of the discipline table after seed data:
Screenshot 2026-02-17 at 9 06 58 PM

After reverting migration:
Screenshot 2026-02-17 at 9 06 10 PM

🏕️ (Optional) Future Work / Notes

The GET endpoint does not correctly reject a discipline that is not found:
Screenshot 2026-02-15 at 5 02 08 PM

Copy link
Collaborator

@SamNie2027 SamNie2027 left a comment

Choose a reason for hiding this comment

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

There are errors when running the migration because you are attempting to recreate things that were present in the previous migrations; please remove the lines that try to recreate the tables that aren't the disciplines.

@rayyanmridha rayyanmridha marked this pull request as ready for review February 18, 2026 02:15
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.

Delete old init Migration and regenerate Discipline, + postman endpoint testing and seeding tests

2 participants

Comments