-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Correctly set the organization num repos #11339
Correctly set the organization num repos #11339
Conversation
Correctly set the organization num repos to the number of accessible repos for the user Fix go-gitea#11194 Signed-off-by: Andrew Thornton <[email protected]>
Signed-off-by: Andrew Thornton <[email protected]>
Signed-off-by: Andrew Thornton <[email protected]>
Signed-off-by: Andrew Thornton <[email protected]>
@zeripath can we add a test for OrgRepoNum somewhere? |
Presumably there is something in OK I've added the test. I've done it in the declarative style with reusable component tests. |
@6543 presumably this is going to need a |
Signed-off-by: Andrew Thornton <[email protected]>
Signed-off-by: Andrew Thornton <[email protected]>
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.
🎉
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.
have to look to all functions
Codecov Report
@@ Coverage Diff @@
## master #11339 +/- ##
==========================================
+ Coverage 43.84% 43.93% +0.09%
==========================================
Files 613 613
Lines 87229 87236 +7
==========================================
+ Hits 38245 38328 +83
+ Misses 44262 44177 -85
- Partials 4722 4731 +9
Continue to review full report at Codecov.
|
* Correctly set the organization num repos Correctly set the organization num repos to the number of accessible repos for the user Fix go-gitea#11194 Signed-off-by: Andrew Thornton <[email protected]> * as per @lunny Signed-off-by: Andrew Thornton <[email protected]> * attempt to fix mssql Signed-off-by: Andrew Thornton <[email protected]> * Update models/user.go * Explicit columns Signed-off-by: Andrew Thornton <[email protected]> * Add test and fix 0 counted orgs Signed-off-by: Andrew Thornton <[email protected]> * remove orgname from api Signed-off-by: Andrew Thornton <[email protected]>
Correctly set the organization num repos to the number of
accessible repos for the user
Fix #11194
Fix #11331
Signed-off-by: Andrew Thornton [email protected]