stakwork / sphinx-tribes-frontend

Frontend for sphinx tribes and bounties. The bounty platform that pays out in bitcoin. Sign up with Sphinx Chat, complete a bounty, and earn bitcoin! Go to our website for available bounties.
https://community.sphinx.chat/bounties
12 stars 42 forks source link

Add Phases to Feature [feature-add-phases-to-feature] #525

Closed fvalentiner closed 4 months ago

fvalentiner commented 4 months ago

Context

In the recently created Workspace page and Feature page where we need to add a list of phases.

Bounty: https://community.sphinx.chat/bounty/1965

Design

Clone this branch https://github.com/stakwork/sphinx-tribes-frontend/tree/feature-add-phases-to-feature

Remember to use the components from Elastic UI when ever posible https://eui.elastic.co/

This will be the full Feature page

image

Your job for this issue is to create the 'Phases' part of the page. Here are the specifics. page: /features/{feature_uuid}

image

Backend Endpoints:

Get list of phases for feature GET /features/{feature_uuid}/phase

Get phase by uuid GET /features/{feature_uuid}/phase/{phase_uuid}

Create or Update Phase POST /features/phase { uuid: '{phase_uuid}', feature_uuid: '{feature_uuid, name: ' {phase name} ', priority: integer }

Delete Phase DELETE /features/{feature_uuid}/phase/{phase_uuid}

Acceptance Criteria

Here is an example unit test

Here is an example component test

MuhammadUmer44 commented 4 months ago

@fvalentiner please assign me

saithsab877 commented 4 months ago

@fvalentiner Could you please assign me?

MirzaHanan commented 4 months ago

@fvalentiner I can help?

MuhammadUmer44 commented 4 months ago

HI @fvalentiner , @elraphty I am trying to create a phase, but I am getting a 401 (Unauthorized) error because @elraphty put a check in the createPhase handler function.

image

When I am creating a phase, I am passing a {feature_uuid} that is necessary for getting the feature's phases, but due to the above check, I am getting a 401 error.

image

fvalentiner commented 4 months ago

@MuhammadUmer44 feature_uuid is the uuid of the [feature] not the [phase]. That id should exist. You are creating a phase for a particular feature. You create the phase from the feature page that already has a feature_uuid so use that value to create the new phase

MuhammadUmer44 commented 4 months ago

@MuhammadUmer44 feature_uuid is the uuid of the [feature] not the [phase]. That id should exist. You are creating a phase for a particular feature. You create the phase from the feature page that already has a feature_uuid so use that value to create the new phase

@fvalentiner, yes you are right

Ekep-Obasi commented 4 months ago

@fvalentiner please assign me