-
-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
Disable Stars config option #14653
Disable Stars config option #14653
Conversation
I think the "watching repos" is a nice func anyway :) ... just if we going to add it, can we add a prifacy opt-out add too? like it exist for |
I used the same settings as "Starred Repositories". Should that be hidden as well? The setting is also ignored for Repositories, Following, and Followed. This seems like a separate PR, but I'm happy to open one to fix that. |
Codecov Report
@@ Coverage Diff @@
## master #14653 +/- ##
==========================================
- Coverage 42.21% 42.21% -0.01%
==========================================
Files 767 767
Lines 81624 81652 +28
==========================================
+ Hits 34458 34468 +10
- Misses 41531 41547 +16
- Partials 5635 5637 +2
Continue to review full report at Codecov.
|
@6543 did you still want me to make the change to put this behind the |
@kdumontnu I think we can seperate it ... |
2638218
to
89c84a7
Compare
8bf3a61
to
53f8d2b
Compare
53f8d2b
to
842ed0f
Compare
@kdumontnu sorry that it wont get reviewed and merged right away... just for you as an info: gitea is in feature freeze at the moment ... |
af2360c
to
50adeb0
Compare
OK I think this looks fine - I don't quite understand why the feature needs disabling but the PR does do everything needed to disable the feature completely. |
Excellent! I understand the hesitation, but generally speaking for private projects "stars" provide very little value and can add unnecessary complexity for users. |
@kdumontnu just reaching out to you here to let you know I sent a PM to your discord, hoping you saw it :) |
Please resolve conflicts |
@kdumontnu can you "update branch" please? |
🚀 |
If
DISABLE_STARS = true
remove the option to star repositoriesThe "Starred Repositories" tab in user profile is replaced with "Watched Repositories"
This is helpful for enterprise/private accounts where the ability to "star" repositories is not very useful and creates confusion compared to "watching".
Note: I'm working on a separate PR to toggle Watch status in the Profile repo list