Sudheesh-Student Profile Educational Progress

[SudheeshTD]

Description

Related PRS (if any):

This Backend PR is related to the #4420 Frontend PR.
To test this feature you need to checkout both backend and frontend PR #4420.
…

Main changes explained:

1. • GET /student/profile
2. • PUT /student/profile
3. • GET /student/profile/subject/:id

…

How to test:

1. check into current branch and the frontend branch #4420

2. do npm install and ... to run this PR locally

3. Clear site data/cache

4. Log in as the user specified in video, or message me in the slack, for details. Use only that user to see data in the student profile page.

5. Follow the frontend PR to follow along in testing.

Screenshots or videos of changes:

video2260393348.mp4

Note:

Include the information the reviewers need to know.

[Anusha-Gali]

Hi Sudeesh,

I have reviewed your PR locally, few comments:
1 - it would be great if you could mention under what are they stored in the db as one of them was Tasks in requirement and you have stored it in studenttask and couldn't locate Students and StudentProgress in DB.
2 - i was not able to retrieve any response for all subject id's except '68def3a5f0844c6916607a94' in the API call: ../student/profile/subject/:id

[SudheeshTD]

Hi @Anusha-Gali,
Can you please elaborate your first comment? I could not understand it. For this task I had manually added 3 users 1 for student, one for teacher one for member. And interlinked them such that when we login using that userid, all the details would be visible, or else it will just view the basic details that your current account has.

[Anusha-Gali]

Hi @Anusha-Gali, Can you please elaborate your first comment? I could not understand it. For this task I had manually added 3 users 1 for student, one for teacher one for member. And interlinked them such that when we login using that userid, all the details would be visible, or else it will just view the basic details that your current account has.

Hi Sudheesh, i was talking about the point - "DB Fields required", i couldn't find them in mongoDB and the one's i found have different name when compared to the suggested one's

[SudheeshTD]

Hi @Anusha-Gali, Can you please elaborate your first comment? I could not understand it. For this task I had manually added 3 users 1 for student, one for teacher one for member. And interlinked them such that when we login using that userid, all the details would be visible, or else it will just view the basic details that your current account has.

Hi Sudheesh, i was talking about the point - "DB Fields required", i couldn't find them in mongoDB and the one's i found have different name when compared to the suggested one's

Hi @Anusha-Gali ,
For the DB fields required, I have not created a new models for everything as the core models that was created initially for the student profile had all the required details. And the requirement didn't specify the relationships of the models with other modals, so I did not create a new one.

And for reference, Student is taken from the UserProfile Model, subjects is taken from subjects model from Database, Student tasks is taken from educationtasks model, and the student progress is within the Get Student profile controller.

Can you test the feature now, I have updated the code for getSubjectDetails.

[Anusha-Gali]

Hi Sudeesh,

I ran the PR again and this is what i am getting in the terminal and postman. Do let me know why am i not getting the same result as you are.

[SudheeshTD]

Hi @Anusha-Gali ,

Can you please check now, the previous commit didn't have the changes, so you were not able to see the modifications. Can you Please check now.

[Anusha-Gali]

1. 4420

This is the only thing i get in postman when i requested the response on the same student id you used

[SudheeshTD]

1. 4420

This is the only thing i get in postman when i requested the response on the same student id you used

Hello @Anusha-Gali ,

I am able to test it still without any issue, after doing a fresh pull. I wanted to check if the pull you did was the most recent one.
Would you mind doing a fresh pull and test again?

video1067019561.mp4

[Anusha-Gali]

Hi @SudheeshTD ,

Thank you so much for the video. Please find attached the video of me testing and i am still not able to. I can maybe try cloning directly and testing but in the mean time, i think it's best if someone else also tries to review the PR.

1908.mp4

[SudheeshTD]

Hi @SudheeshTD ,

Thank you so much for the video. Please find attached the video of me testing and i am still not able to. I can maybe try cloning directly and testing but in the mean time, i think it's best if someone else also tries to review the PR.

Hi @Anusha-Gali,

There is a small changes you have to do when logging in.
Please use the student@test.com user and its corresponding password as shown in video.
This feature currently works for that user itself, as I have added that user with the student permission.
Please let me know once you login using that user.

[sonarqubecloud]

Quality Gate passed

Issues
5 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[Anusha-Gali]

Hi Sudheesh,

Thank you for looking into the code and helping me out. I am now able to receive the API responses.