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

Add project column choice option on issue sidebar #30617

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

a1012112796
Copy link
Member

example:
 test

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Apr 20, 2024
@pull-request-size pull-request-size bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Apr 20, 2024
@github-actions github-actions bot added modifies/translation modifies/go Pull requests that update Go code modifies/templates This PR modifies the template files modifies/js labels Apr 20, 2024
@lunny lunny added this to the 1.23.0 milestone Apr 22, 2024
@lunny
Copy link
Member

lunny commented Apr 23, 2024

A change comment need to be created when switching successfully.

models/project/board.go Outdated Show resolved Hide resolved
models/project/issue.go Outdated Show resolved Hide resolved
@a1012112796
Copy link
Member Author

A change comment need to be created when switching successfully.

That's another feature which is out of the topic of current pull request...

Signed-off-by: a1012112796 <[email protected]>
@lunny
Copy link
Member

lunny commented Apr 23, 2024

A change comment need to be created when switching successfully.

That's another feature which is out of the topic of current pull request...

No, I don't think so. It should be created with the same database transaction.

@a1012112796
Copy link
Member Author

A change comment need to be created when switching successfully.

That's another feature which is out of the topic of current pull request...

No, I don't think so. It should be created with the same database transaction.

see #30703, it's not a small change like what you see...

@yp05327
Copy link
Contributor

yp05327 commented Apr 30, 2024

Considering multi project support, maybe using modal later is a good choice.

@lunny lunny mentioned this pull request May 7, 2024
13 tasks
@lunny
Copy link
Member

lunny commented May 28, 2024

Please resolve the conflicts, sorry the rename PR results in conflicts.

Signed-off-by: a1012112796 <[email protected]>
Signed-off-by: a1012112796 <[email protected]>
routers/web/repo/projects.go Outdated Show resolved Hide resolved
Signed-off-by: a1012112796 <[email protected]>
Signed-off-by: a1012112796 <[email protected]>
@lunny lunny changed the title add project board choice option on issue sidebar Add project column choice option on issue sidebar Jun 3, 2024
@@ -28,6 +28,25 @@ func (issue *Issue) LoadProject(ctx context.Context) (err error) {
return err
}

func (issue *Issue) LoadProjectIssue(ctx context.Context) (err error) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why not use LoadProject directly?

@@ -58,17 +58,17 @@ func (Column) TableName() string {
}

// NumIssues return counter of all issues assigned to the column
func (c *Column) NumIssues(ctx context.Context) int {
func (c *Column) NumIssues(ctx context.Context) (int64, error) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The function is used by template/projects/view.tmpl, you need also to change that place.

}
defer committer.Close()

num, err := toColumn.NumIssues(ctx)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not right. If you update two columns, then they will have the same sorting.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. modifies/go Pull requests that update Go code modifies/js modifies/templates This PR modifies the template files modifies/translation size/L Denotes a PR that changes 100-499 lines, ignoring generated files. topic/projects
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants