Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

View Milestone returns 404 if user cannot write issue/pulls #6331

Closed
1 task done
jolheiser opened this issue Mar 14, 2019 · 0 comments · Fixed by #6328
Closed
1 task done

View Milestone returns 404 if user cannot write issue/pulls #6331

jolheiser opened this issue Mar 14, 2019 · 0 comments · Fixed by #6328
Labels

Comments

@jolheiser
Copy link
Member

Description

Viewing a milestone results in a 404 for all non-write users. Anyone should be able to view the milestone's issues/pulls.

I believe the issue is here:

m.Group("/milestone", func() {
m.Get("/:id", repo.MilestoneIssuesAndPulls)
}, reqRepoIssuesOrPullsWriter, context.RepoRef())

Relevant insight from @lunny:

I think that maybe another bug but after you change that to reqRepoIssuesOrPullsReader, you also need to change the UI to disable Edit milestone and change status of issues or pulls if user haven't reqRepoIssuesOrPullsWriter.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants