Commits at 3ece9d5794420f9f11b2d0628dfe1ca83d24a357
3ece9d57Simplify `IsVendor` (#19626)
The changes in this file were upstreamed directly into go-enry as https://github.com/go-enry/go-enry/pull/44
and therefore they are no longer needed.
Gusted
authored at
GitHub
comitted at
a7f52684Prevent NPE when checking repo units if the user is nil (#19625)
CheckRepoUnitUser should tolerate nil users.
Fix #19613
Signed-off-by: Andrew Thornton <art27@cantab.net>
Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>zeripath
authored at
GitHub
comitted at
0483ee8fAdd "Reference" section to Issue view sidebar (#19609)
* Add "Reference" section to Issue view sidebar
* Removed unneeded tailored CSS, added Fomantic UI classes
* Removed tailored CSS in favour of HTML-only approach
Kindly provided by @Gusted
* Added i18 translation key for reference link string
* Corrected spelling of "References:" to "Reference:" for reference link text
* Removed `Issue.ReferenceLink` in favour of a local template variableneon
authored at
GitHub
comitted at
b536b651GetFeeds must always discard actions with dangling repo_id (#19598)
* GetFeeds must always discard actions with dangling repo_id
See https://discourse.gitea.io/t/blank-page-after-login/5051/12
for a panic in 1.16.6.
* add comment to explain the dangling ID in the fixture
* loadRepoOwner must not attempt to use a nil action.Repo
* make fmt
Co-authored-by: Loïc Dachary <loic@dachary.org>singuliere
authored at
GitHub
comitted at
04fc4b7eCall MultipartForm.RemoveAll when request finishes (#19606)
wxiaoguang
authored at
GitHub
comitted at
7b089c46Remove `RequireHighlightJS` field, update plantuml example. (#19615)
wxiaoguang
authored at
GitHub
comitted at
f034ee6cPullService lock via pullID (#19520)
* lock pull on git&db actions ...
* add TODO notes
* rename prQueue 2 prPatchCheckerQueue
* fmt6543
authored at
GitHub
comitted at
e933f314Add health check endpoint (#18465)
* chore: add health check endpoint
docs: update document about health check
fix: fix up Sqlite3 ping. current ping will success even if the db file is missing
fix: do not expose privacy information in output field
* refactor: remove HealthChecker struct
* Added `/api/healthz` to install routes.
This was needed for using /api/healthz endpoint in Docker healthchecks,
otherwise, Docker would never become healthy if using healthz endpoint
and users would not be able to complete the installation of Gitea.
* Update modules/cache/cache.go
* fine tune
* Remove unnecessary test code. Now there are 2 routes for installation (and maybe more in future)
Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>
Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>
Co-authored-by: 6543 <6543@obermui.de>
Co-authored-by: Marcos de Oliveira <marcossantos@furb.br>ttys3
authored at
GitHub
comitted at
3114cd30Only check for non-finished migrating task (#19601)
* Only check for non-finished migrating task
- Only check if a non-finished migrating task exists for a mirror before
fetching the mirror details from the database.
- Resolves #19600
- Regression: #19588
* Clarify functionGusted
authored at
GitHub
comitted at
9c04da37Make .cs highlighting legible on dark themes. (#19604)
Eekle
authored at
GitHub
comitted at
2d40a4a4Fix broken TR on cherrypick page (#19599)
- `repo.filter_branch` isn't a translation key so use
`repo.pulls.filter_branch` which has the correct translation.Gusted
authored at
GitHub
comitted at
cc3016f7Use correct context in `routers/web` (#19597)
- Don't use the `APIContext` in the web routers.Gusted
authored at
GitHub
comitted at
92f139d0Use for a repo action one database transaction (#19576)
... more context
(part of #9307)6543
authored at
GitHub
comitted at
730420b6Only set CanColorStdout / CanColorStderr to true if the stdout/stderr is a terminal (#19581)
wxiaoguang
authored at
GitHub
comitted at
982b726bDon't fetch Mirror when it's migrating (#19588)
- When a repository is still being migrated, don't try to fetch the
Mirror from the database. Instead skip it. This allows to visit
repositories that are still being migrated and were configured to be
mirrored.
- Resolves #19585
- Regression: #19295
Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>Gusted
authored at
GitHub
comitted at
b7abb31bMove user password verification after checking his groups on ldap auth (#19587)
In case the binded user can not access its own attributes.
Signed-off-by: Gwilherm Folliot <gwilherm55fo@gmail.com>
Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>Gwilherm Folliot
authored at
GitHub
comitted at
772ad761Fix some slice problems (incorrect slice length) (#19592)
Lunny Xiao
authored at
GitHub
comitted at
c8ec2261Fix sending empty notifications (#19589)
- Don't send empty notifications on read notifications API.Gusted
authored at
GitHub
comitted at
438646e0Handle the error of a missing blob object fix #19530 (#19552)
* Handle the error of a missing blob object
* Show error in logs
* as per @zeripath
* Add missing error check
* Add missing error check
* Update compare.go
* Use formal code
* Update compare.go
Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>
Co-authored-by: 6543 <6543@obermui.de>
Co-authored-by: techknowlogick <techknowlogick@gitea.io>99rgosse
authored at
GitHub
comitted at
c18d8d69Remove legacy `+build:` constraint (#19582)
Go 1.17 and later use modern `//go:build` constraints, the old `// +build:` constraints should be removed.wxiaoguang
authored at
GitHub
comitted at
e2a3f3d2Federation: return useful statistic information for nodeinfo (#19561)
Add statistic information for total user count, active user count, issue count and comment count for `/nodeinfo`6543
authored at
GitHub
comitted at
509d8112Upgrade required git version to 2.0 (#19577)
* Upgrade required git version to 2.0
* update documentwxiaoguang
authored at
GitHub
comitted at
0ba3ada8add smtp password to install page (#17564)
* add smtp password to install page
* Update routers/install/install.go
Co-authored-by: zeripath <art27@cantab.net>
Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>Vladimir Smagin
authored at
GitHub
comitted at
71bafa02ignore DNS error when doing migration allow/block check (#19566)
Co-authored-by: 6543 <6543@obermui.de>wxiaoguang
authored at
GitHub
comitted at
d8905cb6Dont overwrite err with nil & rename PullCheckingFuncs to reflect there usage (#19572)
- dont overwrite err with nil unintentionaly
- rename CheckPRReadyToMerge to CheckPullBranchProtections
- rename prQueue to prPatchCheckerQueue
from #9307
Co-authored-by: delvh <dev.lh@web.de>6543
authored at
GitHub
comitted at
3725fa28Improve UI on mobile (#19546)
Start making the mobile experience not painful and be actually usable. This contains a few smaller changes to enhance this experience.
- Submit buttons on the review forms aren't columns anymore and are now allowed to be displayed on one row.
- The label/milestone & New Issue buttons were given each own row even tough, there's enough place to do it one the same row. This commit fixes that.
- The issues+Pull tab on repo's has a third item besides the label/milestone & New Issue buttons, the search bar. On desktop there's enough place to do this on one row, for mobile it isn't, currently it was using for each item a new row. This commits fixes that by only giving the searchbar a new row and have the other two buttons on the same row.
- The notification table will now be show a scrollbar instead of overflow.
- The repo buttons(Watch, Star, Fork) on mobile were showing quite big and the SVG wasn't even displayed on the same line, if the count of those numbers were too high it would even overflow. This commit removes the SVG, as there isn't any place to show them on the same row and allows them to have a new row if the counts of those buttons are high.
- The admin page can show you a lot of interesting information, on mobile the System Status + Configuration weren't properly displayed as the margin's were too high. This commit fixes that by reducing the margin to a number that makes sense on mobile.
- Fixes to not overflow the tables but instead force them to be scrollable.
- When viewing a issue or pull request, the comments aren't full-width but instead 80% and aligned to right, on mobile this is a annoyance as there isn't much width to begin with. This commits fixes that by forcing full-width and removing the avatars on the left side and instead including them inline in the comment header.Gusted
authored at
GitHub
comitted at
ac6c3384Add API to check if team has repo access (#19540)
* Add API to check if team has repo access
* Add test caseqwerty287
authored at
GitHub
comitted at