Non-sense "You pushed on branch ...." notification #11188

Closed
opened 2025-11-02 09:30:16 -06:00 by GiteaMirror · 5 comments
Owner

Originally created by @wxiaoguang on GitHub (Jul 9, 2023).

Originally assigned to: @dsseng on GitHub.

Description

Reproduce step:

  1. Create a branch
  2. Create a PR
  3. Merge the PR
  4. Go to the repo home

Now, these 2 notifications are non-sense.

  • For "main" branch: yes, it is changed.
  • For "root-patch-1": it has been merged.

image

Gitea Version

1.21

Originally created by @wxiaoguang on GitHub (Jul 9, 2023). Originally assigned to: @dsseng on GitHub. ### Description Reproduce step: 1. Create a branch 2. Create a PR 3. Merge the PR 4. Go to the repo home Now, these 2 notifications are non-sense. * [x] For "main" branch: yes, it is changed. * [ ] For "root-patch-1": it has been merged. ![image](https://github.com/go-gitea/gitea/assets/2114189/da3da3be-80b1-472c-bfa1-3be2dd265da5) ### Gitea Version 1.21
GiteaMirror added the type/bug label 2025-11-02 09:30:16 -06:00
Author
Owner

@silverwind commented on GitHub (Jul 9, 2023):

Default branch should never notify imho.

@silverwind commented on GitHub (Jul 9, 2023): Default branch should never notify imho.
Author
Owner

@hiifong commented on GitHub (Jul 9, 2023):

Default branch should never notify imho.

agree

@hiifong commented on GitHub (Jul 9, 2023): > Default branch should never notify imho. agree
Author
Owner

@dsseng commented on GitHub (Jul 9, 2023):

I want to work on this. If I won't submit a fix within a week, consider the task free again

@dsseng commented on GitHub (Jul 9, 2023): I want to work on this. If I won't submit a fix within a week, consider the task free again
Author
Owner

@dsseng commented on GitHub (Jul 10, 2023):

Oops, apparently lunny also took this, but only made it ignore default branch. My PR does also exclude branches with merged PRs. Let me rebase it

@dsseng commented on GitHub (Jul 10, 2023): Oops, apparently lunny also took this, but only made it ignore default branch. My PR does also exclude branches with merged PRs. Let me rebase it
Author
Owner

@wxiaoguang commented on GitHub (Aug 17, 2023):

Close because I am not tracking this problem, maybe some have been fixed.

@wxiaoguang commented on GitHub (Aug 17, 2023): Close because I am not tracking this problem, maybe some have been fixed.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/gitea#11188