Skip to content
Snippets Groups Projects

Api visible projects

1 unresolved thread

What does this MR do?

Add a new /projects/visible API endpoint.

Are there points in the code the reviewer needs to double check?

No.

Why was this MR needed?

The /projects endpoint only returned projects the user was explicitly a member of.

What are the relevant issue numbers?

Closes #19361 (closed), #3119 (closed).

Screenshots (if relevant)

N/A

Does this MR meet the acceptance criteria?

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
175 175 end
176 176 end
177 177
178 describe 'GET /projects/visible' do
179 let(:public_project) { create(:project, :public) }
180
181 it 'returns the projects viewable by the user' do
182 get api('/projects/visible', user3)
183 expect(response).to have_http_status(200)
184 expect(json_response).to be_an Array
185 expect(json_response.map { |project| project['id'] }).to contain_exactly(project.id, project2.id, project4.id)
  • username-removed-443319 Added ~164274 ~13884 labels

    Added ~164274 ~13884 labels

  • How's this coming along?

  • @ben.boeckel: do you need help with the spec question above? I can take that on if you don't have time, just let me know!

  • I haven't gotten time to get back to this and I'm not sure when I'll get back to it, but if you have pointers or even want to take it over, that'd be fine with me.

  • Mentioned in merge request !6681 (merged)

  • Thanks @ben.boeckel! 🚀 I've created a new MR at !6681 (merged) - hopefully it will get into %8.13!

    /cc @bbodenmiller

  • username-removed-443319 Status changed to closed

    Status changed to closed

  • Mentioned in commit 1be15162

  • Please register or sign in to reply
    Loading