Skip to content

Load Issue attributes for API call (#6122)#6185

Merged
techknowlogick merged 1 commit into
go-gitea:release/v1.7from
jolheiser:6056_backport
Feb 26, 2019
Merged

Load Issue attributes for API call (#6122)#6185
techknowlogick merged 1 commit into
go-gitea:release/v1.7from
jolheiser:6056_backport

Conversation

@jolheiser

Copy link
Copy Markdown
Member

Backport from #6122

Signed-off-by: jolheiser <john.olheiser@gmail.com>
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Feb 26, 2019
@lunny lunny added the type/bug label Feb 26, 2019
@lunny lunny added this to the 1.7.3 milestone Feb 26, 2019
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Feb 26, 2019
@techknowlogick techknowlogick merged commit d602ba5 into go-gitea:release/v1.7 Feb 26, 2019
@jolheiser jolheiser deleted the 6056_backport branch February 26, 2019 02:06
@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
project-mirrors-bot-tu Bot pushed a commit to project-mirrors/forgejo-as-gitea-fork that referenced this pull request Oct 11, 2025
…gitea#9045)

Fixes go-gitea#6185.

## Checklist

### Tests

- I added test coverage for Go changes...
  - [x] in their respective `*_test.go` for unit tests.
  - [x] in the `tests/integration` directory if it involves interactions with a live Forgejo server.

I also tested the changes manually.

### Documentation

- [ ] I created a pull request [to the documentation](https://codeberg.org/forgejo/docs) to explain to Forgejo users how to use this change.
- [x] I did not document these changes and I do not expect someone else to do it.

### Release notes

- [ ] I do not want this change to show in the release notes.
- [x] I want the title to show in the release notes with a link to this pull request.
- [ ] I want the content of the `release-notes/<pull request number>.md` to be be used for the release notes instead of the title.

Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/9045
Reviewed-by: Earl Warren <earl-warren@noreply.codeberg.org>
Co-authored-by: Antonin Delpeuch <antonin@delpeuch.eu>
Co-committed-by: Antonin Delpeuch <antonin@delpeuch.eu>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants