bridge-in-tech-backend icon indicating copy to clipboard operation
bridge-in-tech-backend copied to clipboard
verified publisher icon anitab-org

Feat: create GET /mentorship_relations API endpoint

Open mtreacy002 opened this issue 5 years ago • 5 comments

Description

As a developer, I need create GET /mentorship_relations API endpoint, so that the user can see all the requests for the mentorship relation they have received/sent.

Mocks

NA

Acceptance Criteria

Update [Required]

  • [ ] add GET /mentorship_relations API endpoint
  • [ ] prepare all the possible http responses:
    • Ok (200): when the request is sent successfully (this will depend on completion of issue #161)
    • Unauthorized (401): when the token is expired, invalid or missing
    • Internal Server Error (500): when either MS or BIT backend server is facing internal server error.

Enhancement to Update [Optional]

NA

Definition of Done

  • [ ] All of the required items are completed.
  • [ ] Approval by 1 mentor.

Estimation

2-3 days

mtreacy002 avatar Sep 29 '20 09:09 mtreacy002

@mtreacy002 i would like to work on this issue

khushishikhu avatar Sep 30 '20 17:09 khushishikhu

Sure, @khushishikhu . I'll assign you to this. Let me know if you need help 😉

mtreacy002 avatar Sep 30 '20 23:09 mtreacy002

@mtreacy002 please review my PR

khushishikhu avatar Oct 04 '20 05:10 khushishikhu

hey @mtreacy002, I am able to create an endpoint POST /mentorship_relation/send_request which successfully establishes the relationship. If this issue is open and if you say, I can try to add other endpoints in the namespace.

decon-harsh avatar Jan 25 '21 22:01 decon-harsh

@decon-harsh , this issue was already assigned to @khushishikhu and she had submitted a PR (#168) for this. Although, currently there is a block to the issue (#161). As you noticed, we need the POST feature to work first before we can successfully send GET information on mentorship relation. Currently no one is working on issues #161 (I assigned myself but yet to have time to work on it), so, would you like to work on that issue #161 instead?

mtreacy002 avatar Jan 26 '21 21:01 mtreacy002