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

Update to Xorm v0.7.9 #8355

Closed
wants to merge 21 commits into from
Closed

Conversation

guillep2k
Copy link
Member

As title
Should fix #8337

guillep2k and others added 21 commits September 14, 2019 00:18
Update to github.com/lafriks/xormstore@v1.3.0 (go-gitea#8317)
* [arc-green] white on hover for active menu items

Actually, hovered active menu elements are in an unreadable black text colour.
This should be changed to white.

* [arc-green] white on hover for active menu items
Actually, hovered active menu elements are in an unreadable black text colour.
This should be changed to white.

Now editing less files and commiting with `make css`.
…tc.) (go-gitea#7756)

* Add API for Repo Advanced Settings of wiki and issue tracker

Signed-off-by: David Svantesson <davidsvantesson@gmail.com>

* Add some integration tests for tracker and wiki settings through API

* Should return StatusUnprocessableEntity in case of invalid API values.

* Add tests for invalid URLs for external tracker and wiki.

* Do not set inital values if they are default of type

* Make issue tracker and wiki units separate structures in Repository API structure.

Signed-off-by: David Svantesson <davidsvantesson@gmail.com>

* Fix comment of structures

Signed-off-by: David Svantesson <davidsvantesson@gmail.com>

* Rewrite API to use struct for setting tracker and wiki settings.

* LetOnlyContributorsTrackTime -> AllowOnlyContributorsToTrackTime
@guillep2k
Copy link
Member Author

Sorry, I didn't see #8354

@guillep2k guillep2k closed this Oct 2, 2019
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Oct 2, 2019
@codecov-io
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (master@0d93430). Click here to learn what that means.
The diff coverage is 93.2%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #8355   +/-   ##
=========================================
  Coverage          ?   41.75%           
=========================================
  Files             ?      496           
  Lines             ?    65585           
  Branches          ?        0           
=========================================
  Hits              ?    27386           
  Misses            ?    34685           
  Partials          ?     3514
Impacted Files Coverage Δ
routers/repo/webhook.go 1.38% <0%> (ø)
routers/routes/routes.go 82.27% <100%> (ø)
models/repo.go 48.18% <100%> (ø)
routers/api/v1/repo/repo.go 68.22% <95.58%> (ø)

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 0d93430...ca0db2e. Read the comment docs.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants