search

stakwork / sphinx-tribes

Backend for sphinx tribes and bounties. The bounty platform 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
33 stars 60 forks source link

Modify Features endpoint and add delete feature #1639

Closed fvalentiner closed 5 months ago

fvalentiner commented 5 months ago

Context

Step 1 Modify the endpoint url from

/features/forworkspace/{workspace_uuid}

to

workspaces/{workspace_uuid}/features

Step 2 Add endpoint for DELETE /features

Design

  1. clone branch https://github.com/stakwork/sphinx-tribes/tree/feature-add-features-to-workspaces-modify-delete

  2. Make sure you are running posgresql locally

  3. Create file .env with the following content PORT=5005 DATABASE_URL=postgres://postgres:asdf@localhost:5432/ebdb?sslmode=disable

Change DATABASE_URL to connect to your local DB (it could be an empty db)

  1. Run "go run ." to run the backend API that connects to the DB and creates the objects
  2. Open a new terminal window and run "npx cypress run" this starts to run the tests image

Your job is to create the routes, handler functions and db changes in order for the test to pass. If you find any issues or typos in the cypress tests please let the creator of the issue know.

Before committing your changes make sure the cypress tests pass.

Acceptance Criteria

The bounty for this is: https://community.sphinx.chat/bounty/1926

jordan-ae commented 5 months ago

I can help

fvalentiner commented 5 months ago

I can help

Great! What's you user here? https://community.sphinx.chat/p

fvalentiner commented 5 months ago

@jordan-ae

jordan-ae commented 5 months ago

@fvalentiner its Mandela on Sphinx

MirzaHanan commented 5 months ago

Hi @fvalentiner, may I handle this task if @jordan-ae doesn't wish to work on it?

MahtabBukhari commented 5 months ago

Hi @jordan-ae are you working on this task? @fvalentiner if @jordan-ae working on it then its great otherwise I am awailable for this task as i have no any other task.

MahtabBukhari commented 5 months ago

@fvalentiner Could i take this one if @jordan-ae not want to continue with that?

MahtabBukhari commented 5 months ago

Hi @fvalentiner @elraphty Could you please confirm is this enpoint is in used at frontend side image

Becuase Cypress tests are going to be pass only this endpoint as I change the url cypress tests are going to be failed.

MahtabBukhari commented 5 months ago

@fvalentiner @elraphty Could you please update from frontend side becuase this endpoint is used at frontend side due to which when i change this url then cypress are going to be failed "/features/forworkspace/{workspace_uuid}"

fvalentiner commented 5 months ago

@MahtabBukhari leave the old endpoint for now

MahtabBukhari commented 5 months ago

@fvalentiner I have implemented the new endpoint and old one is also present. The benefit is that when we update the url from front-end then no cypress issue Will accor and we can delete the old endpoint when needed later.