Skip to content

Fix error 500 on organization dashboard page#150

Merged
lunny merged 1 commit intogo-gitea:masterfrom
lunny:lunny/update_xorm_for_bug_fix
Nov 11, 2016
Merged

Fix error 500 on organization dashboard page#150
lunny merged 1 commit intogo-gitea:masterfrom
lunny:lunny/update_xorm_for_bug_fix

Conversation

@lunny
Copy link
Member

@lunny lunny commented Nov 11, 2016

Fix the behaviour of the function Count in XORM

This will replace #147. So #147 could be closed.

@thibaultmeyer
Copy link
Contributor

LGTM

@thibaultmeyer thibaultmeyer added this to the 1.0.0 milestone Nov 11, 2016
@thibaultmeyer thibaultmeyer added the type/enhancement An improvement of existing functionality label Nov 11, 2016
@codecov-io
Copy link

Current coverage is 3.03% (diff: 100%)

Merging #150 into master will not change coverage

@@            master      #150   diff @@
========================================
  Files           33        33          
  Lines         8096      8096          
  Methods          0         0          
  Messages         0         0          
  Branches         0         0          
========================================
  Hits           246       246          
  Misses        7830      7830          
  Partials        20        20          

Powered by Codecov. Last update e2aa991...e44329a

@thibaultmeyer thibaultmeyer added the issue/critical This issue should be fixed ASAP. If it is a PR, the PR should be merged ASAP label Nov 11, 2016
@andreynering
Copy link
Contributor

LGTM

@lunny lunny merged commit a8c6698 into go-gitea:master Nov 11, 2016
@lunny lunny deleted the lunny/update_xorm_for_bug_fix branch November 11, 2016 16:41
@thibaultmeyer
Copy link
Contributor

@lunny Maybe you have to port this fix on the real XORM repository too

@lunny
Copy link
Member Author

lunny commented Nov 12, 2016

Of course I already did that

@tboerger tboerger added the lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. label Nov 29, 2016
ethantkoenig pushed a commit to ethantkoenig/gitea that referenced this pull request Apr 29, 2017
ethantkoenig added a commit to ethantkoenig/gitea that referenced this pull request Apr 29, 2017
@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

issue/critical This issue should be fixed ASAP. If it is a PR, the PR should be merged ASAP lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/enhancement An improvement of existing functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants