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(integration): update GitHub scopes to read/write repository issues #709

Merged
merged 2 commits into from
Oct 7, 2024

Conversation

jvallesm
Copy link
Collaborator

@jvallesm jvallesm commented Oct 7, 2024

Because

  • GitHub component's TASK_CREATE_ISSUE didn't work with an OAuth connection.

This commit

  • Updates the OAuth token scopes to allow issue creation / view.
  • Additionally, updates the ListPipelinesByConnectionID endpoint to return only non-deleted pipelines.

@jvallesm jvallesm self-assigned this Oct 7, 2024
@jvallesm jvallesm marked this pull request as ready for review October 7, 2024 11:21
@jvallesm jvallesm merged commit c1cf4ce into main Oct 7, 2024
12 checks passed
@jvallesm jvallesm deleted the fix/jvallesm/update-gh-scope branch October 7, 2024 13:34
jvallesm pushed a commit that referenced this pull request Oct 8, 2024
🤖 I have created a release *beep* *boop*
---


##
[0.42.0-beta](v0.41.0-beta...v0.42.0-beta)
(2024-10-08)


### Features

* **document:** repair pdf with libreoffice
([#683](#683))
([b6a738f](b6a738f))
* **integration:** add identity to OAuth connections
([#699](#699))
([e2e9237](e2e9237))
* merge worker into main process and optimize graceful shutdown
mechanism
([#713](#713))
([69b64a0](69b64a0))
* **universalai:** add instill credit function for universal ai
component
([#680](#680))
([9ce10b5](9ce10b5))
* use explicit parameter for target namespace when cloning pipeline
([#711](#711))
([ab2e3a6](ab2e3a6))
* **web:** fix bug and add mock users' behaviours by chromedp
([#701](#701))
([ca312fe](ca312fe))


### Bug Fixes

* **component:** use kebab-case for icon name
([#703](#703))
([cd587e2](cd587e2))
* **document:** catch the error if the bbox is out of boundary
([#718](#718))
([4fe03e6](4fe03e6))
* fix the component render error was not being returned
([#707](#707))
([931f067](931f067))
* fix the wrong target namespace when cloning pipeline
([#706](#706))
([75a757b](75a757b))
* **integration:** update GitHub scopes to read/write repository issues
([#709](#709))
([c1cf4ce](c1cf4ce))
* **text:** fix the bug about the list position
([#712](#712))
([49aed28](49aed28))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Status: No status
Development

Successfully merging this pull request may close these issues.

3 participants