pull review state bug #12239

Closed
opened 2025-11-02 10:03:02 -06:00 by GiteaMirror · 2 comments
Owner

Originally created by @6543 on GitHub (Dec 19, 2023).

Originally assigned to: @6543 on GitHub.

gitea: v1.21, v1.20, ... and older

reproduce: ( task | actor)

  1. pull to protected branch to USER_A | USER_B
  2. request review of USER_A | USER_B
  3. Review (ack/rej) | USER_A
  4. Push to pull change | USER_B
  5. Dismiss review | USER_A

-> review request can not be deleted
-> if "dismis stale reviews" is set, step 5. is not needed and gitea break it itself

Originally created by @6543 on GitHub (Dec 19, 2023). Originally assigned to: @6543 on GitHub. gitea: **v1.21**, **v1.20**, ... and older **reproduce:** ( task | actor) 1. pull to protected branch to USER_A | USER_B 2. request review of USER_A | USER_B 3. Review (ack/rej) | USER_A 4. Push to pull change | USER_B 5. Dismiss review | USER_A -> review request can not be deleted -> if "dismis stale reviews" is set, step 5. is not needed and gitea break it itself
GiteaMirror added the type/bug label 2025-11-02 10:03:02 -06:00
Author
Owner

@6543 commented on GitHub (Dec 19, 2023):

6a725b6f9c/models/issues/review.go (L463)
6a725b6f9c/models/issues/review.go (L482)

can we just use the xorm builder !!!!!!!

@6543 commented on GitHub (Dec 19, 2023): https://github.com/go-gitea/gitea/blob/6a725b6f9cde28862869befb9d2b101d9e342427/models/issues/review.go#L463 https://github.com/go-gitea/gitea/blob/6a725b6f9cde28862869befb9d2b101d9e342427/models/issues/review.go#L482 can we just use the xorm builder !!!!!!!
Author
Owner

@github-actions[bot] commented on GitHub (Mar 1, 2024):

Automatically locked because of our CONTRIBUTING guidelines

@github-actions[bot] commented on GitHub (Mar 1, 2024): Automatically locked because of our [CONTRIBUTING guidelines](https://github.com/go-gitea/gitea/blob/main/CONTRIBUTING.md#issue-locking)
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/gitea#12239