Skip to content

Workaround: Finding repository when it is null in PR#10481

Closed
rvillablanca wants to merge 1 commit into
go-gitea:release/v1.11from
rvillablanca:fix/panic-owner-name
Closed

Workaround: Finding repository when it is null in PR#10481
rvillablanca wants to merge 1 commit into
go-gitea:release/v1.11from
rvillablanca:fix/panic-owner-name

Conversation

@rvillablanca
Copy link
Copy Markdown
Contributor

This should fix a panic when retrieving the repository owner name. I know this is not a definitive solution but serves as a workaround for release 1.11

Issue #10479

@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Feb 26, 2020
@6543
Copy link
Copy Markdown
Member

6543 commented Feb 26, 2020

pleace resolve conflict

@jolheiser
Copy link
Copy Markdown
Member

jolheiser commented Feb 26, 2020

This was already fixed by #10408

Thank you for the interest and contribution, though! 😃

@rvillablanca
Copy link
Copy Markdown
Contributor Author

Yeah I see, thanks guys. When could we get this fix ?

@jolheiser
Copy link
Copy Markdown
Member

This will be included in 1.11.2 which should be coming hopefully soon. Just cleaning up some other bugs.

@jolheiser jolheiser closed this Feb 26, 2020
@rvillablanca rvillablanca deleted the fix/panic-owner-name branch February 26, 2020 15:07
@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

lgtm/need 1 This PR needs approval from one additional maintainer to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants