mirror of
https://codeberg.org/forgejo/forgejo.git
synced 2026-07-25 02:48:05 +00:00
30b5ea66bc
**Backport:** https://codeberg.org/forgejo/forgejo/pulls/10927
Continuation of #10598
Closes #7254
Implement repo granular watch selection
I adopted a lot of the frontend from #10598 while redoing the entire back-end. I propose this new model:
- Remove the WatchMode enum.
- Add the WatchSelection struct, which represents the granular selection. Notice that there is no `watching` bool. I tried very hard to keep the structure as simple and redundancy-free as possible. Therefore, people not watching a repo at all either don't have a watch record or one with an entirely unselected WatchSelection struct.
- Add the WatchSource enum. It replaces the WatchMode enum and has a single purpose: determine whether a watch was explicitly or automatically initiated.
Notice that replacing this
```go
And("`watch`.mode<>?", WatchModeDont).
```
with this is correct:
```go
And(
builder.Or(
builder.Eq{"`watch`.watch_selection_issues": true},
builder.Eq{"`watch`.watch_selection_pull_requests": true},
builder.Eq{"`watch`.watch_selection_releases": true},
),
).
```
That's because there are four modes: dont, none, auto and normal. When `<>` with dont, we look for auto and normal, because there are no records with none. Therefore, the old code looks for records that indicate watching. The code I replaced this with does so, too, just more granular.
Also notice that I've prepared a future `user preset` in a few places. See below for a little more info on that.
## Next PR
I plan to continue working on this. I want to implement a `user preset` option. The user sets that `user preset` in her settings and may use them in any repo.
<details>
- rename account settings to account and notifications
- user preset (always use this preset for newly accessible repos (according to AutoWatchOnChanges and AutoWatchNewRepos))
</details>
## Further PRs
- make api able to granular watch
- move (email) notifications to new notifications tab
Co-authored-by: 0ko <0ko@noreply.codeberg.org>
Co-authored-by: Gusted <postmaster@gusted.xyz>
Co-authored-by: pat-s <patrick.schratz@gmail.com>
Reviewed-on: #10927
Reviewed-by: Gusted <gusted@noreply.codeberg.org>
(cherry picked from commit
|
||
|---|---|---|
| .. | ||
| ModerationFeatures | ||
| PrivateIssueProjects | ||
| TestActivateUserEmail | ||
| TestAddTeamReviewRequest | ||
| TestGetUnmergedPullRequestsByHeadInfoMax | ||
| TestGetUsedForUser | ||
| TestPackagesGetOrInsertBlob | ||
| TestParseCommitWithSSHSignature | ||
| TestPrivateRepoProjects | ||
| TestTwoFactorWithPasswordChange | ||
| access.yml | ||
| access_token.yml | ||
| action.yml | ||
| action_artifact.yml | ||
| action_run.yml | ||
| action_run_job.yml | ||
| action_runner.yml | ||
| action_runner_token.yml | ||
| action_task.yml | ||
| action_task_output.yml | ||
| action_user.yml | ||
| attachment.yml | ||
| branch.yml | ||
| collaboration.yml | ||
| comment.yml | ||
| commit_status.yml | ||
| commit_status_index.yml | ||
| email_address.yml | ||
| follow.yml | ||
| forgejo_blocked_user.yml | ||
| gpg_key.yml | ||
| hook_task.yml | ||
| issue.yml | ||
| issue_assignees.yml | ||
| issue_index.yml | ||
| issue_label.yml | ||
| issue_user.yml | ||
| issue_watch.yml | ||
| label.yml | ||
| language_stat.yml | ||
| lfs_meta_object.yml | ||
| milestone.yml | ||
| mirror.yml | ||
| notice.yml | ||
| notification.yml | ||
| oauth2_application.yml | ||
| oauth2_authorization_code.yml | ||
| oauth2_grant.yml | ||
| org_user.yml | ||
| project.yml | ||
| project_board.yml | ||
| project_issue.yml | ||
| protected_tag.yml | ||
| public_key.yml | ||
| pull_request.yml | ||
| reaction.yml | ||
| release.yml | ||
| renamed_branch.yml | ||
| repo_redirect.yml | ||
| repo_topic.yml | ||
| repo_transfer.yml | ||
| repo_unit.yml | ||
| repository.yml | ||
| review.yml | ||
| star.yml | ||
| stopwatch.yml | ||
| system_setting.yml | ||
| team.yml | ||
| team_repo.yml | ||
| team_unit.yml | ||
| team_user.yml | ||
| topic.yml | ||
| tracked_time.yml | ||
| two_factor.yml | ||
| user.yml | ||
| user_open_id.yml | ||
| user_redirect.yml | ||
| watch.yml | ||
| webauthn_credential.yml | ||
| webhook.yml | ||