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

Refactor issue subscription #8737

Closed
6543 opened this issue Oct 29, 2019 · 8 comments
Closed

Refactor issue subscription #8737

6543 opened this issue Oct 29, 2019 · 8 comments

Comments

@6543
Copy link
Member

6543 commented Oct 29, 2019

gitea version: until latest

current behaviour:

  • if you watch a repo you get notify on issue change ALWAYS
  • if you you not watch a repo you can get notify when subscribe to specific issue (DB table: issue_watch)

what i would expect:

  • if i watch a repo but unsubscribe on a issue I do NOT get notify
@guillep2k
Copy link
Member

Well, #7657 addresses this kind of problem but for repos by setting different types of subscription instead of on/off, auto, normal and don't (none being no record at all).

@stale
Copy link

stale bot commented Dec 28, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs during the next 2 weeks. Thank you for your contributions.

@stale stale bot added issue/stale and removed issue/stale labels Dec 28, 2019
@6543
Copy link
Member Author

6543 commented Dec 28, 2019

note for me: look at DB.tables: watch issue_watch

@6543
Copy link
Member Author

6543 commented Dec 28, 2019

notify.action()
  make( map userNotifyed[] <- ignore user)

  get mentionedUsers
   -> if have read acces && !userNotifyed -> notify
   -> add to userNotifyed
 get issue subscriber
   -> if have read acces && !userNotifyed  -> notify
   -> add to userNotifyed

 get repo watcher
   -> if have read acces && !userNotifyed  -> notify
   -> add to userNotifyed

  if action == create {
    if !Doer.Subscribed_to_repo && !Doer.Subscribed_to_Issue && !IssueWatchModeDont && settings-allow-this {
       auto subscribe to issue
    }
  } 

@6543
Copy link
Member Author

6543 commented Dec 28, 2019

👀 models.notification.go L77

@stale
Copy link

stale bot commented Feb 27, 2020

This issue has been automatically marked as stale because it has not had recent activity. I am here to help clear issues left open even if solved or waiting for more insight. This issue will be closed if no further activity occurs during the next 2 weeks. If the issue is still valid just add a comment to keep it alive. Thank you for your contributions.

@stale stale bot added the issue/stale label Feb 27, 2020
@6543
Copy link
Member Author

6543 commented Feb 27, 2020

yep this issue can be closed ... pull's are on the way and have there own issues :)

@6543 6543 closed this as completed Feb 27, 2020
@6543
Copy link
Member Author

6543 commented Feb 27, 2020

(#10478, #10475, [#10473 or #9535], #10401, ...)

@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.
Projects
None yet
Development

No branches or pull requests

2 participants