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.

Sign up for GitHub

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

Jump to bottom

Moving project panels results in 500 GetProjectBoard: project board does not exist #29853

Closed
silverwind opened this issue Mar 16, 2024 · 4 comments · Fixed by #29874
Closed

Moving project panels results in 500 GetProjectBoard: project board does not exist #29853

silverwind opened this issue Mar 16, 2024 · 4 comments · Fixed by #29874
Labels
issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented topic/projects type/bug

Comments

@silverwind
Copy link
Member

silverwind commented Mar 16, 2024 •
edited

Description

  1. Create Project
  2. Create two panels in it
  3. Drag-and-drop the panels, swapping them. Observe 500 error from PUT request:
...web/repo/projects.go:526:checkProjectBoardChangePermissions() [E] GetProjectBoard: project board does not exist [id: 0]
...eb/routing/logger.go:102:func1() [I] router: completed PUT /silverwind/symlink-test/projects/1/0 for 127.0.0.1:56716, 500 Internal Server Error in 30.2ms @ repo/projects.go:546(repo.EditProjectBoard)

Reproduced on https://try.gitea.io/silverwind/symlink-test/projects/3308.

Gitea Version

main
@silverwind silverwind changed the title Moving project panel horizontally results in 500 GetProjectBoard: project board does not exist Moving project panels results in 500 GetProjectBoard: project board does not exist Mar 16, 2024
@silverwind silverwind mentioned this issue Mar 16, 2024
Merged
@yardenshoham yardenshoham added the issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented label Mar 16, 2024
@lunny
Copy link
Member

lunny commented Mar 17, 2024

I cannot reproduce it on my local main branch.

@lng2020 lng2020 mentioned this issue Mar 17, 2024
Closed
@lng2020
Copy link
Member

lng2020 commented Mar 17, 2024

@lunny It only happens when sorting the default project board

@lunny lunny added this to the 1.21.9 milestone Mar 18, 2024
silverwind added a commit to silverwind/gitea that referenced this issue Mar 18, 2024
@silverwind
Prevent dropping onto project column 0
92ec5a5 
Fixes: go-gitea#29853
Fixes: go-gitea#29870
@silverwind silverwind mentioned this issue Mar 18, 2024
Closed
@denyskon denyskon mentioned this issue Mar 18, 2024
Merged
@lunny
Copy link
Member

lunny commented Mar 19, 2024

This will be resolved by #29874 but that cannot be backport to v1.21. Should we have another PR to do a quick fix for v1.21?

@silverwind
Copy link
Member Author

It's not a critical thing, so I would not backport.

silverwind added a commit that referenced this issue Mar 19, 2024
@silverwind @wxiaoguang
Forbid jQuery .css and refactor all usage (#29852)
fa10061 
Tested all functionality. There is a [pre-existing
bug](#29853) when moving a
project panels which is not caused by this refactoring.

---------

Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>
@lunny lunny removed this from the 1.21.9 milestone Mar 20, 2024
DennisRasey pushed a commit to DennisRasey/forgejo that referenced this issue Mar 26, 2024
@silverwind @earl-warren
Forbid jQuery .css and refactor all usage (#29852)
8c3d7c0 
Tested all functionality. There is a [pre-existing
bug](go-gitea/gitea#29853) when moving a
project panels which is not caused by this refactoring.

---------

Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>
(cherry picked from commit fa100618c4b644346bf5666f92d33dce0747d0a2)
@6543 6543 closed this as completed in e516018 Mar 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented topic/projects type/bug
Projects
None yet
Milestone
No milestone
4 participants
@lunny @silverwind @yardenshoham @lng2020