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

Fix work bench issue data query #3354

Merged
merged 4 commits into from
Dec 15, 2021

Conversation

shuofan
Copy link
Contributor

@shuofan shuofan commented Dec 13, 2021

What type of this PR

bugfix

What this PR does / why we need it:

Fix work bench issue data query

Which issue(s) this PR fixes:

Specified Reviewers:

/assign @sfwn

ChangeLog

Language Changelog
🇺🇸 English
🇨🇳 中文

Need cherry-pick to release versions?

Add comment like /cherry-pick release/1.0 when this PR is merged.

For details on the cherry pick process, see the cherry pick requests section under CONTRIBUTING.md.

@erda-bot erda-bot requested a review from sfwn December 13, 2021 08:20
@shuofan shuofan added bugfix dop devops plaoform labels Dec 13, 2021
@codecov
Copy link

codecov bot commented Dec 13, 2021

Codecov Report

Merging #3354 (ec689e8) into master (371d720) will increase coverage by 0.02%.
The diff coverage is 25.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3354      +/-   ##
==========================================
+ Coverage   17.83%   17.86%   +0.02%     
==========================================
  Files        1386     1386              
  Lines      144314   144384      +70     
==========================================
+ Hits        25736    25789      +53     
+ Misses     115902   115897       -5     
- Partials     2676     2698      +22     
Impacted Files Coverage Δ
modules/dop/services/issue/issue.go 2.92% <0.00%> (ø)
modules/dop/services/workbench/workbench.go 89.00% <100.00%> (ø)
modules/monitor/apm/topology/topology.go 27.54% <0.00%> (-0.05%) ⬇️
modules/core-services/endpoints/notify_group.go 0.00% <0.00%> (ø)
...odules/core/monitor/log/query/log.query.service.go 16.93% <0.00%> (ø)
modules/core-services/services/project/project.go 16.06% <0.00%> (+0.20%) ⬆️
modules/msp/apm/service/apm.service.service.go 69.45% <0.00%> (+10.10%) ⬆️


func (client *DBClient) issueExpiryStatusQuery(req apistructs.WorkbenchRequest) *gorm.DB {
sql := client.Debug().Table("dice_issues").Joins(joinState)
sql = sql.Where("deleted = 0").Where("assignee = ? AND dice_issue_state.belong IN (?)", req.Assignees, req.StateBelongs)
Copy link
Member

Choose a reason for hiding this comment

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

assignee = or IN ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

change to IN

@sfwn
Copy link
Member

sfwn commented Dec 15, 2021

/approve

@erda-bot erda-bot merged commit 8ed0350 into erda-project:master Dec 15, 2021
@shuofan shuofan deleted the bug/work-bench-data branch December 15, 2021 03:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

3 participants