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.

Sign up for GitHub

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

Jump to bottom

Prevent 500 is head repo does not have PullRequest unit in IsUserAllo… #20839

Merged
merged 5 commits into from
Aug 18, 2022
Merged

Prevent 500 is head repo does not have PullRequest unit in IsUserAllo… #20839

merged 5 commits into from
Aug 18, 2022

Commits on Aug 17, 2022

  1. Prevent 500 is head repo does not have PullRequest unit in IsUserAllo… 

    …wedToUpdate

Some repositories do not have the PullRequest unit present in their configuration
and unfortunately the way that IsUserAllowedToUpdate currently works assumes
that this is an error instead of just returning false.

This PR simply swallows this error allowing the function to return false.

Fix go-gitea#20621

Signed-off-by: Andrew Thornton <art27@cantab.net>
    @zeripath
    zeripath committed Aug 17, 2022
    Configuration menu
    Copy the full SHA
    b93d5b7 View commit details
    Browse the repository at this point in the history

Commits on Aug 18, 2022

  1. Merge branch 'main' into fix-20621-no-err-in-IsUserAllowedToUpdate

    @lunny
    lunny authored Aug 18, 2022
    Configuration menu
    Copy the full SHA
    fdd742b View commit details
    Browse the repository at this point in the history

  2. Merge branch 'main' into fix-20621-no-err-in-IsUserAllowedToUpdate

    @lunny
    lunny authored Aug 18, 2022
    Configuration menu
    Copy the full SHA
    561e84e View commit details
    Browse the repository at this point in the history

  3. Merge branch 'main' into fix-20621-no-err-in-IsUserAllowedToUpdate

    @lafriks
    lafriks authored Aug 18, 2022
    Configuration menu
    Copy the full SHA
    1803e6a View commit details
    Browse the repository at this point in the history

  4. Merge branch 'main' into fix-20621-no-err-in-IsUserAllowedToUpdate

    @lafriks
    lafriks authored Aug 18, 2022
    Configuration menu
    Copy the full SHA
    c0192bb View commit details
    Browse the repository at this point in the history