Skip to content

Fix print oauth url without flush #548

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mcgr0g
Copy link
Member

@mcgr0g mcgr0g commented Apr 10, 2025

Description

PR for #544

Non-technical explanation

Make it possible to passing oauth link through Power User for DBT plugin.

Release notes

( ) This is not user-visible or docs only and no release notes are required.
( ) Release notes are required, please propose a release note for me.
( ) Release notes are required, with the following suggested text:

* Fix print oauth url without flush. ({issue}`544`)

Copy link

cla-bot bot commented Apr 10, 2025

Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Kuzin Vlad.
This is most likely caused by a git client misconfiguration; please make sure to:

  1. check if your git client is configured with an email to sign commits git config --list | grep email
  2. If not, set it up using git config --global user.email [email protected]
  3. Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails

@mcgr0g mcgr0g marked this pull request as draft April 10, 2025 21:52
@mcgr0g mcgr0g force-pushed the 544-print-with-flush branch from 7876adb to 0483617 Compare April 10, 2025 22:44
Copy link

cla-bot bot commented Apr 10, 2025

Thank you for your pull request and welcome to the Trino community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. Continue to work with us on the review and improvements in this PR, and submit the signed CLA to [email protected]. Photos, scans, or digitally-signed PDF files are all suitable. Processing may take a few days. The CLA needs to be on file before we merge your changes. For more information, see https://github.com/trinodb/cla

@mcgr0g mcgr0g marked this pull request as ready for review April 10, 2025 22:46
@mcgr0g mcgr0g marked this pull request as draft April 10, 2025 23:25
@mcgr0g mcgr0g marked this pull request as ready for review April 17, 2025 21:06
@mcgr0g mcgr0g self-assigned this Apr 17, 2025
@mcgr0g mcgr0g force-pushed the 544-print-with-flush branch from 0483617 to 1c698b9 Compare April 17, 2025 21:20
@cla-bot cla-bot bot added the cla-signed label Apr 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

1 participant