@anjisvj could you please update the documentation with this
@piumal1999 could you please review and merge this ASAP
1 Like
Is ‘update state of a program’ a non-reversible action? @YohanAvishke
Yes for the current stage. But in case a requirement arises it’s flexible to change
1 Like
The directory structure of test controllers are not similar to the directory structure of controllers. It is possible to use the same test controller file for two separate controllers. But i think it’s better if we can keep the same directory structure in test controllers and controllers to make it less complex.
So i created an issue for that:
Previous structure:
├── controller
│ ├── MenteeControllerTest.java
│ ├── MentorControllerTest.java
│ └── ProgramControllerTest.java
└── service
New structure that i suggest:
├── controller
│ ├── admin
│ │ ├── MenteeControllerTest.java
│ │ ├── MentorControllerTest.java
│ │ └── ProgramControllerTest.java
│ └── ProgramControllerTest.java
└── service
Sent a PR for that. Please review and merge
2 Likes
Awsome job @piumal1999 
With the previous discussions, we have found some minor bugs and issues in the scholarX backend that needed to fixed. The requirements are mentioned below.
- Add a JSON response payload to DELETE endpoints
- Add a column for mentor application
- Rename questionnaire to the prerequisite
- Change the program state MENTEE_APPLY to MENTEE_APPLICATION
I created issues for these in the github repository. I’m working on these issues and I’ll use this thread for the updates and discussion
Related links:
1 Like
Fixed the issue with this pull request:
Please review and merge
1 Like