Skip to content

chore(integration): pass out of date from backend#119082

Merged
joseph-sentry merged 12 commits into
masterfrom
joey--gh-integration-nudge-be
Jul 7, 2026
Merged

chore(integration): pass out of date from backend#119082
joseph-sentry merged 12 commits into
masterfrom
joey--gh-integration-nudge-be

Conversation

@joseph-sentry

@joseph-sentry joseph-sentry commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

like the slack integration we want to have nudges / buttons for the
github integration so users can easily be notified that some installation
is out of date, and so they can update it

part of this is updating the serializer for integrations to pass an
"out of date" field to let the frontend know that the integration needs
to be updated based on logic that runs in the backend that has easy access to
options not exposed through the API

also moving around the slack scope class definition bc it was causing a circular
import issue

logic for commenting on autofix PRs during PR iteration
when tool calls fail potentially due to out of date github app
permissions

logic for passing "warnings" back from group ai autofix GET endpoint
specifically for failed tool calls potentially due to out of date github app
permissions
like the slack integration we want to have nudges / buttons for the
github integration so users can easily be notified that some installation
is out of date, and so they can update it

part of this is updating the serializer for integrations to pass an
"out of date" field to let the frontend know that the integration needs
to be updated based on logic that runs in the backend that has easy access to
options not exposed through the API

also moving around the slack scope class definition bc it was causing a circular
import issue
@joseph-sentry joseph-sentry changed the title Joey gh integration nudge be chore(integration): pass out of date from backend Jul 6, 2026
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Jul 6, 2026

@alexsohn1126 alexsohn1126 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot!

@joseph-sentry joseph-sentry marked this pull request as ready for review July 7, 2026 15:15
@joseph-sentry joseph-sentry requested review from a team as code owners July 7, 2026 15:15
@joseph-sentry joseph-sentry merged commit 338ac05 into master Jul 7, 2026
67 of 68 checks passed
@joseph-sentry joseph-sentry deleted the joey--gh-integration-nudge-be branch July 7, 2026 15:35
joseph-sentry added a commit that referenced this pull request Jul 7, 2026
…119111)

deps on #119082

We already have nudges for updating slack app permissions so we want to
add nudges for the github app as well. This works differently from the
slack app nudges since for the slack app to update the app you go
through the install flow again, but in this case there's a dedicated URL
for updating the github app and going through the install flow again
will not work.

There's also a refactor in here regarding the deletion in the
integration detailed page where it now uses `useMutation` and this is so
we can avoid a race condition when refetching since we changed the stale
time for the config query on that page to 0.

| Label | Screenshot |
|---|---|
| Integration row tooltip | <img width="280" height="162" alt="image"
src="https://github.com/user-attachments/assets/42a94d13-75c7-438c-a6e7-ed1b5038246f"
/> |
| Integration detail alert | <img width="1423" height="729" alt="image"
src="https://github.com/user-attachments/assets/700bb9f7-6059-4422-82be-ea747a3a8c95"
/> |
| After clicking update now button on alert with 1 outdated config |
<img width="1419" height="737" alt="image"
src="https://github.com/user-attachments/assets/17af4852-6f74-4736-b3a9-08c884a3f76a"
/> |
| Integration detail configuration tab | <img width="1398" height="260"
alt="image"
src="https://github.com/user-attachments/assets/11ba4163-7ace-4f33-a1f7-fce3688988f5"
/> |
| Integration detail tooltip | <img width="313" height="217" alt="image"
src="https://github.com/user-attachments/assets/f3e5b6e7-36e9-4ac2-a9f4-056e61061efb"
/> |
| After clicking update now button on integration in integration
detailed view | <img width="1422" height="643" alt="image"
src="https://github.com/user-attachments/assets/3fc4d484-581c-4616-a219-1a1c6d23e93a"
/> |

---------

Co-authored-by: getsantry[bot] <66042841+getsantry[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants