Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

feat: check pipelines' tokens before executing/creating pipeline (#8055) #8069

Merged
merged 17 commits into from
Sep 30, 2024
Merged

feat: check pipelines' tokens before executing/creating pipeline (#8055) #8069

merged 17 commits into from
Sep 30, 2024

Conversation

d4x1
Copy link
Contributor

@d4x1 d4x1 commented Sep 19, 2024 •
edited
Loading

⚠️ Pre Checklist

Please complete ALL items in this checklist, and remove before submitting

  • I have read through the Contributing Documentation.
  • I have added relevant tests.
  • I have added relevant documentation.
  • I will add labels to the PR, such as pr-type/bug-fix, pr-type/feature-development, etc.

Summary

What does this PR do?

Does this close any open issues?

Closes #8017

Screenshots

Include any relevant screenshots here.

Other Information

Any other information that is important to this PR.

@dosubot dosubot bot added size:L This PR changes 100-499 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 Sep 19, 2024
d4x1 and others added 14 commits September 29, 2024 15:41
@d4x1 @mintsweet
feat: check pipelines' tokens before executing/creating pipeline (#8055)
1b9dc10 
* feat(pipeline): check pipelines' tokens before executing/creating pipeline

* fix(framework): fix lint errors

* fix(framework): fix lint errors

* feat(github): check multi tokens when creating pipelines
@d4x1 @mintsweet
feat(project): add token check result in project check API (#8099)
9cdd3b4
@d4x1 @mintsweet
Dev 1 (#8100)
8796873 
* feat(project): add token check result in project check API

* feat(project): update token check API response
@d4x1 @mintsweet
Dev 1 (#8101)
c5daba3 
* feat(project): add token check result in project check API

* feat(project): update token check API response

* fix(project): fix api doc

* fix(project): update project check api doc
@d4x1 @mintsweet
fix(project): update check project api doc
89096fa
@mintsweet
feat: add connection token check before collect data
4a393ed
@d4x1 @mintsweet
feat(blueprint): add new api to check connection tokens (#8102)
dc705a6
@mintsweet
fix: adjust the api for connections token check
8c09e33
@mintsweet
feat: add a component connection-form-modal
9163679
@mintsweet
feat: add a component connection-name
be5ac5b
@mintsweet
fix: show connection form modal when connection token check failed
175f3d6
@mintsweet
refactor: improve the component connection-name
1fa4745
@mintsweet
fix: error check connection token failed list
4942c81
@mintsweet
refactor: extract the function in the panel to bp detail
8bf8f8e
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Sep 30, 2024
@mintsweet mintsweet merged commit 11db8e9 into main Sep 30, 2024
18 of 19 checks passed
@mintsweet mintsweet deleted the feat-8017 branch September 30, 2024 02:30
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@mintsweet mintsweet mintsweet approved these changes

Assignees
No one assigned
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
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Feature][UX] Validate token before running pipelines
2 participants
@d4x1 @mintsweet