Merged
Conversation
Signed-off-by: Majid Ali Ansari <ansmajidali@gmail.com>
Signed-off-by: Majid Ali Ansari <ansmajidali@gmail.com>
There was a problem hiding this comment.
Pull request overview
This PR modernizes the UI across the Django ERP app (ERPNext-style layout + light/dark mode), adds a “Remember me” login flow, and updates several attendance/notification-related UI flows (including CSRF headers on AJAX calls).
Changes:
- Introduces new ERPNext-style base/layout templates and new dashboard pages for staff and students.
- Adds “Remember me” session behavior and session-related settings.
- Updates multiple templates for Bootstrap 5 modal attributes, responsive table scrolling, and notification/attendance AJAX behavior.
Reviewed changes
Copilot reviewed 46 out of 60 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| media/419814426_7030120057025080_6401746063475529990_n.jpg | Adds/updates an image asset. |
| main_app/views.py | Adds “Remember me” session expiry logic after login. |
| main_app/urls.py | Renames the update-attendance POST route path while keeping the same URL name. |
| main_app/templates/student_template/view_books.html | Adds scroll container/min table width for responsiveness. |
| main_app/templates/student_template/student_view_result.html | Adds scroll container/min table width for responsiveness. |
| main_app/templates/student_template/student_view_attendance.html | Makes attendance data container scrollable. |
| main_app/templates/student_template/student_feedback.html | Adds scroll container and truncation styling for long text. |
| main_app/templates/student_template/student_apply_leave.html | Adds scroll container and truncation styling for long text. |
| main_app/templates/student_template/home_content.html | Removes inline Firebase messaging code from the old student home. |
| main_app/templates/student_template/erpnext_student_home.html | New ERPNext-style student dashboard + charts + Firebase messaging code. |
| main_app/templates/staff_template/staff_update_attendance.html | Adds CSRF header handling for AJAX POST calls. |
| main_app/templates/staff_template/staff_feedback.html | Adds scroll container and truncation styling for long text. |
| main_app/templates/staff_template/staff_apply_leave.html | Adds scroll container and truncation styling for long text. |
| main_app/templates/staff_template/home_content.html | Removes inline Firebase messaging code from the old staff home. |
| main_app/templates/staff_template/erpnext_staff_home.html | New ERPNext-style staff dashboard + charts + Firebase messaging code. |
| main_app/templates/registration/password_reset_form.html | Updates reset form to new ERPNext registration base + refreshed markup. |
| main_app/templates/registration/password_reset_email.html | Updates the password reset email content/formatting. |
| main_app/templates/registration/password_reset_done.html | Updates reset-done page to new ERPNext registration base + refreshed markup. |
| main_app/templates/registration/password_reset_confirm.html | Updates reset-confirm page to new ERPNext registration base + refreshed markup. |
| main_app/templates/registration/password_reset_complete.html | Updates reset-complete page to new ERPNext registration base + refreshed markup. |
| main_app/templates/registration/erpnext_base.html | New ERPNext-styled base template for registration flows. |
| main_app/templates/main_app/sidebar_template.html | Adds default-avatar fallback logic and removes sidebar logout item. |
| main_app/templates/main_app/login.html | Redesigns login page UI and adds “Remember me” checkbox. |
| main_app/templates/main_app/erpnext_sidebar.html | New ERPNext-style sidebar template with role-based navigation. |
| main_app/templates/main_app/erpnext_login.html | Adds a separate ERPNext-style login page template. |
| main_app/templates/main_app/erpnext_base.html | Adds an alternate ERPNext base including notification manager + theme toggle. |
| main_app/templates/main_app/base.html | Replaces AdminLTE-style base with ERPNext layout + theme + chart handling. |
| main_app/templates/hod_template/student_notification.html | Updates Bootstrap 5 modal attributes and adds CSRF header + validation for AJAX. |
| main_app/templates/hod_template/student_leave_view.html | Adds scroll container, truncation styling, Bootstrap 5 modal attributes, and CSRF header for AJAX. |
| main_app/templates/hod_template/student_feedback_template.html | Updates Bootstrap 5 modal attributes and adds CSRF header + validation for AJAX. |
| main_app/templates/hod_template/staff_notification.html | Updates Bootstrap 5 modal attributes and adds CSRF header + validation for AJAX. |
| main_app/templates/hod_template/staff_leave_view.html | Adds scroll container, truncation styling, Bootstrap 5 modal attributes, and CSRF header for AJAX. |
| main_app/templates/hod_template/staff_feedback_template.html | Updates Bootstrap 5 modal attributes and adds CSRF header + validation for AJAX. |
| main_app/templates/hod_template/manage_student.html | Adds scroll container/min table width for responsiveness. |
| main_app/templates/hod_template/manage_staff.html | Adds scroll container/min table width for responsiveness. |
| main_app/templates/hod_template/erpnext_home_fixed.html | Adds an ERPNext-style HOD dashboard variant. |
| main_app/templates/hod_template/erpnext_home_content.html | Adds an ERPNext-style HOD dashboard that extends erpnext_base. |
| main_app/student_views.py | Switches student home render target to the new ERPNext student dashboard template. |
| main_app/static/plugins/sweetalert2-theme-bootstrap-4/bootstrap-4.scss | Removes SweetAlert2 Bootstrap 4 theme SCSS asset. |
| main_app/static/plugins/bs-custom-file-input/bs-custom-file-input.min.js.map | Removes bs-custom-file-input sourcemap asset. |
| main_app/static/plugins/bs-custom-file-input/bs-custom-file-input.min.js | Removes bs-custom-file-input minified JS asset. |
| main_app/static/plugins/bs-custom-file-input/bs-custom-file-input.js.map | Removes bs-custom-file-input sourcemap asset. |
| main_app/static/plugins/bs-custom-file-input/bs-custom-file-input.js | Removes bs-custom-file-input JS asset. |
| main_app/static/dist/js/notification-manager.js | Adds a shared NotificationManager wrapper for Firebase Cloud Messaging. |
| main_app/static/dist/css/style.css | Adds a large additional stylesheet (separate from erpnext-style.css). |
| main_app/staff_views.py | Switches staff home render target to the new ERPNext staff dashboard template. |
| college_management_system/settings.py | Adds session settings supporting “Remember me” behavior. |
| README.md | Major README refresh with new formatting, sections, and credentials. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Update