Added result page and book appointment page . by Jk-karunadu · Pull Request #70 · AOSSIE-Org/NeuroTrack

Jk-karunadu · 2025-03-21T11:19:55Z

Closes #32

📝 Description

This PR adds result page of assessment that conducted and therapy goals screen and daily activity that should be followed , below that doctors list to get appointment .

📷 Screenshots or Visual Changes (if applicable)

are in issue #32

✅ Checklist

I have read the contributing guidelines.
I have added tests that prove my fix is effective or that my feature works.
I have added necessary documentation (if applicable).
Any dependent changes have been merged and published in downstream modules.

…ersonal details

…22' into post-assessment

mdmohsin7 · 2025-03-21T15:48:03Z

patient/lib/presentation/assessments/assessment_screen.dart

 import 'package:provider/provider.dart';
+import 'package:patient/presentation/result/result.dart';
+    //  final questions = [
+    //                         {


pls remove comments like this

mdmohsin7 · 2025-03-21T15:49:17Z

patient/lib/presentation/auth/personal_details_screen.dart

+
+     if (_formKey.currentState?.validate() ?? false) {
+
+  if (isAssessmentForChild) {


pls check these conditions code

mdmohsin7 · 2025-03-21T15:49:54Z

patient/lib/presentation/result/book_appointment.dart

+    setState(() => isLoading = true);
+
+    try {
+      await supabase.from("session").insert({


logic should be in provider, pls check the rest of the codebase for reference

mdmohsin7 · 2025-03-21T15:50:48Z

patient/lib/presentation/result/result.dart

+  }
+
+  // Send responses to Groq API for analysis
+  Future<void> analyzeResponses() async {


the goal of this issue was only to design the ui and code it, not to add the logic, pls check again

i just thought of integrating llms to get daily tasks and therapy goals . should i need to remove it.

Yes pls remove it

Context:

mdmohsin7 · 2025-03-21T15:54:12Z

The UI still does not reflect the changes mentioned in #32 (comment), pls check again. Essentially the UI should show the user their percentage of being autistic (and this is not something to be happy about to congratulate, rather the text should be neutral) and a list of therapist with whom they can book a consultation. They should be able to see the details of the therapists upon clicking on any therapist card and also they should be able to select the date and time slot (of 30 mins)

Jk-karunadu · 2025-03-21T17:34:05Z

@mdmohsin7 since details about therapist consist only that much of details :

to increase the deatils about therapist need to add description about the therapist to therapist table , or using some static data to design UI
should i create table or use some static data ?

mdmohsin7 · 2025-03-21T17:42:59Z

Pls use static data for now (through provider)

Jk-karunadu · 2025-03-21T17:45:20Z

@mdmohsin7 okay

…h therapists

…oTrack into post-assessment

Jk-karunadu · 2025-03-21T19:50:59Z

@mdmohsin7 as you said i completed the task using static data , only concentrated on UI ,

Screencast.from.2025-03-22.01-08-28.webm

please review , the may look dull because of i am running it in chrome , but with android it may look good with perfect layout

Jk-karunadu · 2025-03-22T07:08:18Z

@mdmohsin7 Please review this

mdmohsin7 · 2025-03-22T14:02:07Z

@Jk-karunadu can you pls remove the your recommendations section. We won't really have to show it. Only the list of doctors/therapists has to be shown

…screen

…oTrack into post-assessment

Jk-karunadu · 2025-03-23T17:03:48Z

@mdmohsin7 could u please review now

Jk-karunadu · 2025-03-23T17:04:15Z

@mdmohsin7 i removed "your recomendation " section

mdmohsin7 · 2025-03-24T17:35:18Z

patient/lib/presentation/result/result.dart

+  }
+
+  // Send responses to Groq API for analysis
+  Future<void> analyzeResponses() async {


Yes pls remove it

Context:

mdmohsin7 · 2025-03-24T17:36:55Z

patient/lib/presentation/result/result.dart

+    );
+  }
+
+  Widget _buildResultsView() {


Pls move these sub widgets to widgets folder inside of the result folder. They shouldn't be placed in a single file like this

This is still there, pls check

mdmohsin7 · 2025-03-24T17:37:43Z

patient/lib/presentation/result/result.dart

+}
+
+// Placeholder screens (enhanced)
+class TherapyGoalsScreen extends StatelessWidget {


I don't think we need this?

mdmohsin7 · 2025-03-24T17:38:05Z

patient/lib/presentation/result/result.dart

+  }
+}
+
+class DailyActivitiesScreen extends StatelessWidget {


This as well, I don't think it is needed

mdmohsin7 · 2025-03-24T17:52:09Z

@Vivekgupta008 if you are reviewing the PRs then pls make sure that the PR does what the issue mentions and follows good code practises. Otherwise it wouldn't be helpful for me as I'll have to review the PR completely again

…ding view

…oTrack into post-assessment

Jk-karunadu · 2025-03-24T18:19:17Z

@mdmohsin7 please review , i just removes all the section and added only the result part and doctor recomendation list part .

mdmohsin7 · 2025-03-27T15:07:17Z

@Jk-karunadu pls check the review again, not all changes that I requested have been done

Jk-karunadu · 2025-03-27T16:40:31Z

@mdmohsin7 I have checked this now PR consists of the code of result of assessments and doctors least , no extra is added , no llm is integrated, no therapy goals etc things . Only restricted to result of assessments and doctors least.

mdmohsin7

pls check, one change is still there that hasn't been done yet

mdmohsin7 · 2025-03-28T18:36:01Z

patient/lib/presentation/result/result.dart

+    );
+  }
+
+  Widget _buildResultsView() {


This is still there, pls check

mdmohsin7

remaining changes can be done in separate PRs, need this PR merged to continue with other flows

Jk-karunadu and others added 4 commits March 18, 2025 18:28

Add result and therapist details screens, integrate navigation from p…

684446e

…ersonal details

Merge remote-tracking branch 'upstream/assessments-list-screen-issue-…

b72f8e1

…22' into post-assessment

Added result page and appointment booking page

9454c66

Merge branch 'main' into post-assessment

a7db3c9

mdmohsin7 requested changes Mar 21, 2025

View reviewed changes

Jk-karunadu and others added 3 commits March 22, 2025 01:10

feat: add TherapistProvider for fetching and booking appointments wit…

67baa07

…h therapists

Merge branch 'post-assessment' of https://github.com/Jk-karunadu/Neur…

7dd5262

…oTrack into post-assessment

Merge branch 'main' into post-assessment

701d563

Jk-karunadu and others added 3 commits March 23, 2025 22:30

refactor: remove recommendations and next steps sections from result …

c71a8c3

…screen

Merge branch 'post-assessment' of https://github.com/Jk-karunadu/Neur…

1b9fb39

…oTrack into post-assessment

Merge branch 'main' into post-assessment

aa4a8d1

Vivekgupta008 approved these changes Mar 24, 2025

View reviewed changes

mdmohsin7 requested changes Mar 24, 2025

View reviewed changes

Jk-karunadu added 3 commits March 24, 2025 23:47

feat: add result widgets for displaying therapist information and loa…

6318781

…ding view

Merge branch 'post-assessment' of https://github.com/Jk-karunadu/Neur…

827aed5

…oTrack into post-assessment

chore: update dependencies in pubspec.lock to latest versions

6a97d25

mdmohsin7 approved these changes Mar 28, 2025

View reviewed changes

patient/lib/presentation/result/result.dart

);

}

Widget _buildResultsView() {

Copy link

Member

mdmohsin7 Mar 28, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is still there, pls check

mdmohsin7 approved these changes Mar 30, 2025

View reviewed changes

mdmohsin7 merged commit c26fa43 into AOSSIE-Org:main Mar 30, 2025


		if (_formKey.currentState?.validate() ?? false) {

		if (isAssessmentForChild) {

Uh oh!

Conversation

Jk-karunadu commented Mar 21, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

📝 Description

📷 Screenshots or Visual Changes (if applicable)

✅ Checklist

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

mdmohsin7 Mar 24, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

mdmohsin7 commented Mar 21, 2025

Uh oh!

Jk-karunadu commented Mar 21, 2025

Uh oh!

mdmohsin7 commented Mar 21, 2025

Uh oh!

Jk-karunadu commented Mar 21, 2025

Uh oh!

Jk-karunadu commented Mar 21, 2025

Uh oh!

Jk-karunadu commented Mar 22, 2025

Uh oh!

mdmohsin7 commented Mar 22, 2025

Uh oh!

Jk-karunadu commented Mar 23, 2025

Uh oh!

Jk-karunadu commented Mar 23, 2025

Uh oh!

mdmohsin7 Mar 24, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

mdmohsin7 commented Mar 24, 2025

Uh oh!

Jk-karunadu commented Mar 24, 2025

Uh oh!

mdmohsin7 commented Mar 27, 2025

Uh oh!

Jk-karunadu commented Mar 27, 2025

Uh oh!

mdmohsin7 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

mdmohsin7 left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Jk-karunadu commented Mar 21, 2025 •

edited

Loading

mdmohsin7 Mar 24, 2025 •

edited

Loading

mdmohsin7 Mar 24, 2025 •

edited

Loading