Developing ScholarX backend

This thread is to be used for scholarX backend development discussions.

Github repository for backend:

I’m working on “Adding JPA annotations to generate mysql tables”.


Deadline: 2020-08-19T18:29:00Z

Hi @piumal1999 Thanks for working on the feature :heart_eyes: .Can you please create an issue for this.

1 Like

Added the PR. Please review and merge

1 Like

I am working on Admin APIs of ScholarX. I have sent a draft pull request for following tasks.

  • An endpoint to delete a mentee
  • An endpoint to get a list of mentors in a program
  • An endpoint to get update the state of a mentor

I’m not sure whether this is the most suitable way to complete the issue.
@YohanAvishke could you please check it and let me know.


Please review and merge

1 Like

I have checked it using Swagger UI. Btw I think we need to disable CSRF to make the APIs functional. @YohanAvishke is it okay if I add the security config file to my PR and disable csrf?

Sure @piumal1999, add a file named SecurityConfig in the config directory with the changes.
Make sure to create a different commit without squashing.

1 Like

Updated the PR. Please review and merge.

1 Like

@piumal1999 please add test cases to your PR. You can find examples in the above PR. Please ping me if you need any help.

1 Like

Updated the PR with requested changes
Added unit tests for 3 new admin APIs

Created issues for common and user APIs

Ps: Created issues for other APIs also


Created common APIs and unit test for those.

1 Like

Found a bug in this PR. (An infinite loop in GET mentors endpoint)
Fixed the bug with PR#49

Please review and merge

Created User APIs. Please review and merge

I created the mentor APIs with unit tests
But i couldn’t test the changes with a real database

curl --location --request PUT 'program/{programId}/mentees/{menteeId}/status'

In requests similar to above which require programId to identify an element, wouldn’t it be better to add the programId as a query param? Following are some points to support the statement,

  1. Param is an identification parameter, so it’s better to be in the query.
  2. We can simplify the resource paths:
    curl --location --request PUT 'mentees/{menteeId}/status?programId='
  3. This leads to a better and simpler folder structure.


1 Like

Yeah. That’s good. So, shall i update the backend?

Yeah update your PR with the change also change the directory structure accordingly

1 Like

Do we need to add the requested change to the user APIs also?

Pull request for USER API: