-
-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
Add custom helm repo name generated from url #21024
Conversation
routers/web/admin/admin.go
Outdated
@@ -243,6 +243,7 @@ func Config(ctx *context.Context) { | |||
|
|||
ctx.Data["CustomConf"] = setting.CustomConf | |||
ctx.Data["AppUrl"] = setting.AppURL | |||
ctx.Data["GiteaRepoName"] = setting.Domain |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why are you declaring both a context variable and a helper function? One of them ought to be enough and I'd lean to just using Domain
which is set 1 line below this one, so we can remove both the helper and the variable.
Is this PR active? I think it could be easy to get approved: remove all other unnecessary |
@wxiaoguang I created FRESH PR without conflicts #24363 |
Replace helm repo name in Packages to generated from gitea instance site url. Because default "gitea" repo name has many installations in internet but repo name based on url unique for every instance.