Skip to content

Conversation

@ostepan8
Copy link

@ostepan8 ostepan8 commented Jan 20, 2026

ℹ️ Issue

Closes #52
idk why that linked to the other thing... (Code-4-Community#52)

📝 Description

Write a short summary of what you added. Why is it important? Any member of C4C should be able to read this and understand your contribution -- not just your team members.

Briefly list the changes made to the code:

  1. Added CRUD operations for the controller to add admins to disciplines, remove disciplines, and edit them.
  2. Added endpoints to expose these operations to the client side

✔️ Verification

What steps did you take to verify your changes work? These should be clear enough for someone to be able to clone the branch and follow the steps themselves.

I made a bunch of tests and they all pased.

Provide screenshots of any new components, styling changes, or pages.

🏕️ (Optional) Future Work / Notes

Did you notice anything ugly during the course of this ticket? Any bugs, design challenges, or unexpected behavior? Write it down so we can clean it up in a future ticket!

piersolh and others added 30 commits September 14, 2025 14:37
[Issue #16] -  create discipline entity in TypeORM
added testing
removed deprecated on update
adding user authentication by adding provider
deleted old migration file and used just the main one
SamNie2027 and others added 26 commits January 4, 2026 20:51
…lass-validators-on-all-dtos

99 implement stricter class validators on all DTOs
Added a TODO comment to indicate intentional breakage for GitHub Actions.
Added a TODO comment to indicate a temporary change for GitHub Actions.
…utomated-frontend-and-backend-builds

101 GitHub actions ci automated frontend and backend builds
- added controller functions for edit, remove, add
- added endpoints that support the new controller operations
@ostepan8 ostepan8 closed this Jan 20, 2026
@SamNie2027 SamNie2027 deleted the edit-remove-create-discipline-routes branch January 21, 2026 02:12
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.

9 participants