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

Migrate Description #17626 #17692

Merged
merged 4 commits into from
Nov 18, 2021
Merged

Migrate Description #17626 #17692

merged 4 commits into from
Nov 18, 2021

Conversation

99rgosse
Copy link
Contributor

This PR sets the parameter for migrating the description of a repository : solves #17626
Actual behavior is opts.Description is only taken in account

I had to modify services/migrations/gitea_uploader.go too as the parameter wasn't modified if not a set one.

Tested okay on latest dev

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Nov 17, 2021
@99rgosse
Copy link
Contributor Author

Sorry I may have made a mistake when force pushing to the branch, I did not see you added things. Please tell me if I have to change anything

@lunny
Copy link
Member

lunny commented Nov 17, 2021

Sorry I may have made a mistake when force pushing to the branch, I did not see you added things. Please tell me if I have to change anything

Yes, you should rebase the latest main branch. Or you can create a new PR.

@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Nov 18, 2021
@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 Nov 18, 2021
@lunny
Copy link
Member

lunny commented Nov 18, 2021

make L-G-T-M work

@lunny lunny merged commit e1d6559 into go-gitea:main Nov 18, 2021
@lunny
Copy link
Member

lunny commented Nov 18, 2021

Please send backport to v1.15

@99rgosse 99rgosse deleted the migrate-description branch November 19, 2021 10:06
@lunny lunny added the backport/done All backports for this PR have been created label Nov 19, 2021
@wxiaoguang wxiaoguang added this to the 1.16.0 milestone Nov 22, 2021
Chianina pushed a commit to Chianina/gitea that referenced this pull request Mar 28, 2022
Co-authored-by: 6543 <6543@obermui.de>
Co-authored-by: techknowlogick <techknowlogick@gitea.io>
Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>
@go-gitea go-gitea locked and limited conversation to collaborators Apr 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backport/done All backports for this PR have been created 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.

6 participants