Skip to content
This repository has been archived by the owner on Apr 7, 2024. It is now read-only.

GitHub Integrations Improvements #61

Merged
merged 9 commits into from
Jan 15, 2024
Merged

GitHub Integrations Improvements #61

merged 9 commits into from
Jan 15, 2024

Conversation

NicHaley
Copy link
Contributor

More gracefully handles the GitHub installation workflow for when a user goes through the GitHub integration flow and triggers an "Installation Request" on GitHub. This happens in situations where the requesting user does not have admin privileges in GitHub.

Unfortunately, this request flow is not yet well supported by GitHub:
https://github.com/orgs/community/discussions/42351

This means that it is not possible to store the installation on the workspace programmatically. Therefore, as a workaround this solution will send an email to me with the installation id. I can then manually authenticate the request and then update the workspace github integration with the installationId and set the status to installed.

In the future, this solution should be made programatic if GitHub indeed updates the request callback.

Copy link

vercel bot commented Jan 14, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
floe-api ❌ Failed (Inspect) Jan 15, 2024 2:44pm
floe-app ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 15, 2024 2:44pm
floe-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 15, 2024 2:44pm

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant