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

feat: add jira_trace plugin to jira bp #7569

Merged
merged 16 commits into from
Jun 6, 2024
Merged

feat: add jira_trace plugin to jira bp #7569

merged 16 commits into from
Jun 6, 2024

Conversation

abeizn
Copy link
Contributor

@abeizn abeizn commented Jun 4, 2024

Summary

feat: add jira_trace plugin to jira bp

Does this close any open issues?

Closes na

Screenshots

image
image
image

Other Information

Any other information that is important to this PR.

@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. component/plugins This issue or PR relates to plugins pr-type/feature-development This PR is to develop a new feature labels Jun 4, 2024
@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. and removed size:S This PR changes 10-29 lines, ignoring generated files. labels Jun 5, 2024
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:M This PR changes 30-99 lines, ignoring generated files. labels Jun 5, 2024

db := taskCtx.GetDal()
inserter := helper.NewBatchSaveDivider(taskCtx, utils.BATCH_SIZE, rawTableIssueChangelogs, boardId)
inserter := helper.NewBatchSaveDivider(taskCtx, utils.BATCH_SIZE, rawTableIssueChangelogs, scopeIds[0])
Copy link
Contributor

Choose a reason for hiding this comment

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

This doesn't seem correct. It looks like all records will be marked as if they belong to the first scope while not.

BoardId uint64 `json:"boardId"` // 68
LakeBoardId string `json:"lakeBoardId"` // jira:JiraBoard:1:68
Plugin string `json:"plugin"` // jira
ScopeIds []string `json:"scopeId"` // 68
Copy link
Contributor

Choose a reason for hiding this comment

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

json should be renamed to scopeIds as well

klesh
klesh previously approved these changes Jun 6, 2024
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jun 6, 2024
@klesh klesh merged commit 54fe382 into main Jun 6, 2024
10 checks passed
@klesh klesh deleted the jira-bp-add-issue_trace branch June 6, 2024 04:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/plugins This issue or PR relates to plugins lgtm This PR has been approved by a maintainer pr-type/feature-development This PR is to develop a new feature size:L This PR changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants