-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Gad 2 New Commits #1440
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
Gad 2 New Commits #1440
Conversation
Co-authored-by: A Anunaya <[email protected]> Co-authored-by: Aksh Bansal <[email protected]>
@Srivatsa19 please fill the format of the PR, add relevant information and add screenshots + usage video |
Filetracking part seems to work properly, and PR format is followed. Looks good |
I think its ready to merge @prajjwalkapoor |
LGTM, merging. |
FusionIIIT/applications/globals/api/views.py, I talked with the dashboard team, and this change is not supposed to be here. |
Issue that this pull request solves
Added functionality where only logged in users can access the module. The module has been implemented according to the use case diagram
Closes: # (issue number)
Proposed changes
Brief description of what is fixed or changed
Types of changes
Put an
x
in the boxes that applyChecklist
Put an
x
in the boxes that applyScreenshots
https://drive.google.com/drive/folders/1iQBxUOuhojei7qLjph9K1bhximrseokW?usp=sharing
Please attach the screenshots of the changes made in case of change in user interface
Other information
Any other information that is important to this pull request
The designations used in the module are :-
designations_list = ["Junior Engineer", "Executive Engineer (Civil)", "Electrical_AE", "Electrical_JE", "EE", "Civil_AE", "Civil_JE", "Dean (P&D)", "Director", "Accounts Admin", "Admin IWD", "Auditor"]
ajay --> Junior Engineer
dvijay --> Executive Engineer (Civil)
de --> Dean (P&D)
dir --> Director
richard --> Accounts Admin
iwdAdmin --> Admin IWD
kunal --> Auditor
@varun7singh
@ChaudharyRaman
@prajjwalkapoor