Skip to content

Conversation

tboerger
Copy link
Member

@tboerger tboerger commented Jan 2, 2018

That way we are able to add additional custom temmplate content more easily. A good example is https://try.gitea.io where we place a notice at the top, currently we are overwriting the header template and we are lacking changes if the template gets updated.

@tboerger tboerger added type/enhancement An improvement of existing functionality topic/ui Change the appearance of the Gitea UI labels Jan 2, 2018
@tboerger tboerger added this to the 1.4.0 milestone Jan 2, 2018
@lafriks
Copy link
Member

lafriks commented Jan 2, 2018

LGTM

@tboerger tboerger added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Jan 2, 2018
{{template "custom/header" .}}
</head>
<body>
{{template "custom/body_outer_pre" .}}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

custom/templates/custom/ ? :trollface:

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bkcsoft just following the already existing templates custom/header.tmpl and custom/footer.tmpl.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

root@dchi: ~ <o> # tree /var/lib/docker/volumes/demo_gitea/_data/templates /var/lib/docker/volumes/demo_gitea/_data/templates └── base ├── head.tmpl ├── head.tmpl.master ├── head.tmpl.new └── head.tmpl.patch 

:trollface:

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:trollface:

@bkcsoft
Copy link
Member

bkcsoft commented Jan 2, 2018

LG-TM doesn't update on approvals 😢

@tboerger tboerger 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 Jan 2, 2018
@tboerger
Copy link
Member Author

tboerger commented Jan 2, 2018

LG-TM doesn't update on approvals 😢

It updated?

@bkcsoft
Copy link
Member

bkcsoft commented Jan 2, 2018

@tboerger only after commenting :(

@tboerger
Copy link
Member Author

tboerger commented Jan 2, 2018

@tboerger only after commenting :(

Than @appleboy isn't listening to the correct events? Maybe raise an issue on the LGTM repo, or finally replace it with a simple bot :trollface:

@codecov-io
Copy link

Codecov Report

Merging #3286 into master will increase coverage by 0.21%.
The diff coverage is n/a.

Impacted file tree graph

@@ Coverage Diff @@ ## master #3286 +/- ## ========================================== + Coverage 34.6% 34.82% +0.21%  ========================================== Files 277 277 Lines 40263 40263 ========================================== + Hits 13935 14021 +86  + Misses 24283 24183 -100  - Partials 2045 2059 +14
Impacted Files Coverage Δ
models/repo.go 43.38% <0%> (+0.18%) ⬆️
models/repo_indexer.go 51.94% <0%> (+0.97%) ⬆️
models/lfs.go 28.26% <0%> (+2.17%) ⬆️
modules/process/manager.go 81.15% <0%> (+4.34%) ⬆️
modules/lfs/server.go 35.01% <0%> (+14.32%) ⬆️
modules/lfs/content_store.go 43.75% <0%> (+35.93%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a220377...88587af. Read the comment docs.

@bkcsoft bkcsoft merged commit c4a8d4c into go-gitea:master Jan 2, 2018
@tboerger tboerger deleted the additional-template-snippets branch January 2, 2018 10:44
@appleboy
Copy link
Member

appleboy commented Jan 3, 2018

@tboerger @bkcsoft we need to re-enable the gitea repo for receive approval event.

I don't have permission to do this.

https://lgtm.gitea.io/go-gitea

@appleboy
Copy link
Member

appleboy commented Jan 3, 2018

screen shot 2018-01-03 at 11 15 40 am

Please see the webhook setting.

@go-gitea go-gitea locked and limited conversation to collaborators Nov 23, 2020
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. topic/ui Change the appearance of the Gitea UI type/enhancement An improvement of existing functionality

5 participants