-
Notifications
You must be signed in to change notification settings - Fork 982
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
SA-3 : Implemented Use Cases for Each Actor as per UCD #1310
Conversation
* removed conflicts from approve request from dean * change co and coco api working
* use cases init * club approoval request to dean working
* use cases init * club approoval request to dean working * removed conflicts
bug fixed: date foreign key issue resolved in apis
Fixed several APIs for mobile application use
fixed: new club joinee request approval and rejection
fix: multiple votes issue resolved in gymkhana/voting_poll
corrected: club details information on the club budget form & rolled back requirements.txt
fix: packages file edited
fix:updated migrations error
segregated use cases based on actors
fix:fixed gymkhana sidebar buttons according to roles
fixed: type in designations
Congratulations for making your first Pull Request at Fusion!! 🎉 Someone from our team will review it soon. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @Anurag607 please add a demo video and code explanation
And all the migrations files
Thanks!
@varun7singh I have updated the PR description with a demo video. PTAL |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remove the migrations files
fic navigators added
removed migrations
@varun7singh the migrations file has been removed |
There are still some migration files left also please merge into SA3 not main |
@Anurag607 we have some merge conflicts too please resolve them and remove the migrations files use
in gitignore that should do it |
local migrations removed
all migration files removed
@varun7singh I have updated the base branch and removed all the migrations, PTAL. |
@Anurag607 there are multiple conflicts please resolve them |
Use Case Diagram:
Video:
https://drive.google.com/file/d/1WGUPHmgKnbs7yCFFfcyGfkP__qQsYIga/view?usp=sharing