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

Issues Subscription different Types #8852

Closed
5 tasks
6543 opened this issue Nov 6, 2019 · 2 comments
Closed
5 tasks

Issues Subscription different Types #8852

6543 opened this issue Nov 6, 2019 · 2 comments
Labels
type/enhancement An improvement of existing functionality

Comments

@6543
Copy link
Member

6543 commented Nov 6, 2019

Migrate to new Types

current: true (1), false(0)

aim: IssueWatchModeNone, IssueWatchModeNormal, IssueWatchModeDont, IssueWatchModeAuto

ToDo

  • DB migration
  • Update Related code
  • new Function to get Types
  • extend subscription API
    GET /repos/{owner}/{repo}/issues/{index}/subscriptions/{user}
  • ignore issue if unsubscribed but repowatch

Notes

IssueWatchModeNone = not watching / no DB entry
IssueWatchModeNormal = subscribed to issue (manualy)
IssueWatchModeDont = explisit unsubscribe
IssueWatchModeAuto = subscribed by automation (write a comment at this issue)

RepoWatch should respect IssueWatchModeDont in future ...

@6543 6543 mentioned this issue Nov 6, 2019
4 tasks
@lunny lunny added the type/enhancement An improvement of existing functionality label Nov 6, 2019
@6543
Copy link
Member Author

6543 commented Dec 6, 2019

this should close part of #145 (Automatic subscribe on Open/Comment, etc)

@6543
Copy link
Member Author

6543 commented Jan 23, 2020

WIP PR: #9535

@6543 6543 closed this as completed Mar 3, 2020
@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type/enhancement An improvement of existing functionality
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants