chore(integration): pass out of date from backend#119082
Merged
Merged
Conversation
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
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>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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