Skip to content
This repository was archived by the owner on Sep 22, 2025. It is now read-only.

Conversation

@KaustubhK98
Copy link
Collaborator

Description

The exercise module was using a "Grid Layout" to handle the exercise card actions. I replaced the "Grid Layout" with a "Recycler View". Now, the exercise cards will be dynamically added or removed from the adapter and after every action the page will be automatically refreshed. This issue has fixed the bug, which lead to blank spaces on the page and made the UI more responsive. Recycler View is a future proof solution over Grid Layout due to its ability to dynamically handle card actions and scalability.

Todos

  • Tested and working locally
  • Code follows the style guidelines of this project
  • I have performed a self-review of my code
  • Code changes documented
  • Requested review from >= 2 devs on the team (one frontend and one backend recommended)

How to test

  1. Login to app
  2. Go to exercise module page
  3. Select any module and click on "mark as complete" or "save for later" button.
  4. The card should be gone and there shouldn't be any blank space on the page.
  5. The UI is dynamic now and is updated automatically after any changes are made to the exercise cards.

Screenshots and/or Gifs

Uploading Screen Recording 2025-08-10 at 9.24.52 pm.mov…

Associated MS Planner Tasks

Known Issues

There aren't any known issues arising due to this change.

@AryanSharma1017
Copy link
Collaborator

AryanSharma1017 commented Aug 27, 2025

Hey Kaustubh, just check for the build once again by running ./gradlew spotlessapply and then check for the ./gradlew build and then see, I think then all these errors should go.
if not then please have a talk with me and we will see.

@KaustubhK98 KaustubhK98 force-pushed the gm-frontend-kaustubh branch from 5f9a416 to f10fd62 Compare August 29, 2025 09:45
@KaustubhK98
Copy link
Collaborator Author

Hi Aryan,

Spotless build wasn't an issue. There was a Gradle version mismatch, I have fixed the problem on my end and raised another PR. Can you please let me know if you are able to see the new PR. Please try to merge the code now, I am certain that it will get merged this time. Please let me know if there are any problems while merging the new PR.

Thanks!

…est to the doctor. Also, added a check to prevent sending duplicate requests. Added a search bar to search for doctors from the list
@KaustubhK98
Copy link
Collaborator Author

1. Added functionality to assign doctor to a patient from the caretaker's profile:

  • A caretaker can now send a request to a doctor before actually assigning a patient to them.
  • A caretaker can directly assign a patient to the doctor
  • A caretaker can Un-assign or cancel a request sent to a doctor.
  • The app doesn’t allow duplicate requests
  • Doctors can be searched using the search bar.

2. Safely removed the night mode and it's files from the app.

3. Added a function to view the list of patients assigned to a doctor. This function is partially implemented and needs the doctor's portal to work properly.

Known Issues
There aren't any known issues arising due to this change.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants