Updated README.md#27
Conversation
jerrya-code
left a comment
jerrya-code
left a comment
There was a problem hiding this comment.
Sweet! Thanks for making changes and updating the README file!
There was a problem hiding this comment.
Pull request overview
This PR updates project documentation to reflect recent user-facing features (notifications and theme selection) and includes small Android project refactors to align package/layout references.
Changes:
- Updated
README.mdto mention the notification tool and selectable light/dark theme. - Updated
activity_schedule.xml(phone + w600dp)tools:contextto referencefeatures.assignments.ScheduleActivity. - Updated
ScheduleActivity.ktpackage/imports and removed the old root-levelCanvasAPI.kt.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| app/src/main/res/layout/activity_schedule.xml | Updates layout preview context to the new ScheduleActivity package. |
| app/src/main/res/layout-w600dp/activity_schedule.xml | Same tools:context update for tablets. |
| app/src/main/java/com/example/studentproductivityapp/features/assignments/ScheduleActivity.kt | Moves ScheduleActivity into the features.assignments package and cleans up type/import usage. |
| app/src/main/java/com/example/studentproductivityapp/CanvasAPI.kt | Removes the old root-level Canvas API client definitions. |
| README.md | Documents notification system and light/dark mode selection. |
Comments suppressed due to low confidence (1)
app/src/main/java/com/example/studentproductivityapp/features/assignments/ScheduleActivity.kt:36
AppDatabase.Companion.getDatabase(this)is redundant and inconsistent with the rest of the codebase (e.g., AddAssignmentActivity/HomeFragment useAppDatabase.getDatabase(...)). Prefer callingAppDatabase.getDatabase(this)directly for consistency and readability.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| #### Assignment Tracker: Full CRUD operations for student tasks using a local database. | ||
| #### Canvas API Sync: Access Token based retrieval of "To-Do" items from Idaho State University’s Canvas instance (allowed with FERPA according to ISU IT department recommendation) | ||
| #### Interactive Campus Map: Integrated WebView with interactive ISU map capabilities. | ||
| #### PDF Scanner (OCR): Ability to scan documents and extract text to automatically create assignments. | ||
| #### Video Lecture Tool: Transcript search and synchronized video playback. | ||
| #### Notification Tool: Reminds user of assignment due dates in advance |
There was a problem hiding this comment.
The PR title/description indicate a README-only update, but this PR also changes Android layouts, moves ScheduleActivity into a new package, and removes a Kotlin source file. Please update the PR title/description (or split the PR) so reviewers understand the scope and intent of these non-README changes.
Updated Readme to note new notification system and light/dark mode selection.