-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Stack Overflow plugin support for PAT authentication #17157
Stack Overflow plugin support for PAT authentication #17157
Conversation
Signed-off-by: Chris Hawkins <cmdkeen@users.noreply.github.com>
Signed-off-by: Chris Hawkins <cmdkeen@users.noreply.github.com>
Changed Packages
|
Signed-off-by: Chris <cmdkeen@users.noreply.github.com>
Uffizzi Preview |
Signed-off-by: Chris <cmdkeen@users.noreply.github.com>
This PR has been automatically marked as stale because it has not had recent activity from the author. It will be closed if no further activity occurs. If the PR was closed and you want it re-opened, let us know and we'll re-open the PR so that you can continue the contribution! |
Morning! Can this be re-opened, it should still be valid. @emmaindal are you able to help with this? |
Hello @cmdkeen thanks a lot for opening this PR, I'll review it! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great to me, just one small comment before we can get this merged! And sorry for slow response here, we really appreciate your contribution!
Signed-off-by: Chris Hawkins <cmdkeen@users.noreply.github.com>
Signed-off-by: Chris Hawkins <cmdkeen@users.noreply.github.com>
Thanks for taking a look @emmaindal - that should be the config fixed and also clarified that the new URL is different for Teams |
Thank you for contributing to Backstage! The changes in this pull request will be part of the |
Hey, I just made a Pull Request!
Fixes #15770 by adding support for the new PAT authentication for Stack Overflow as per docs.
Deliberately keeping as a minor change - I've not bumped the default API version number and added a new optional parameter. This is because it isn't clear whether the API key approach is still required for other Stack Overflow integrations.
My first PR against Backstage - great documentation but apologies in advance if I've missed anything!
✔️ Checklist
Signed-off-by
line in the message. (more info)