Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

batches: allow refreshing commit signing apps installation#60125

Merged
BolajiOlajide merged 8 commits into
mainfrom
bo/batch-changes-refresh-github-installs
Feb 14, 2024
Merged

batches: allow refreshing commit signing apps installation#60125
BolajiOlajide merged 8 commits into
mainfrom
bo/batch-changes-refresh-github-installs

Conversation

@BolajiOlajide

@BolajiOlajide BolajiOlajide commented Feb 3, 2024

Copy link
Copy Markdown
Contributor

This gives site admins the ability to refresh a commit signing app, instead of waiting for the periodic worker (which runs every 24 hours) to kick of the installation sync.

CleanShot 2024-02-14 at 17 51 06@2x

Test plan

Add a GitHub app on Sourcegraph for Commit Signing.
There should be a Refresh button to refresh installations on the app.

If the app is installed on any new org or account, the installations will be added to the database after the Refresh operation is done.

@BolajiOlajide BolajiOlajide self-assigned this Feb 3, 2024
@cla-bot cla-bot Bot added the cla-signed label Feb 3, 2024
@BolajiOlajide BolajiOlajide force-pushed the bo/batch-changes-refresh-github-installs branch from 5a353f3 to ddd6b31 Compare February 14, 2024 16:54
@BolajiOlajide BolajiOlajide marked this pull request as ready for review February 14, 2024 16:57
@BolajiOlajide BolajiOlajide requested a review from a team February 14, 2024 16:57
Comment thread cmd/frontend/internal/githubapp/resolver.go Outdated
Comment thread client/web/src/enterprise/batches/settings/CommitSigningIntegrationNode.tsx Outdated

@vovakulikov vovakulikov left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

FE review is ok, I see that only BatchChanges codebases adopted spring animations

@BolajiOlajide BolajiOlajide force-pushed the bo/batch-changes-refresh-github-installs branch from e626ee4 to 92decd5 Compare February 14, 2024 17:27
@BolajiOlajide BolajiOlajide enabled auto-merge (squash) February 14, 2024 17:28
@BolajiOlajide BolajiOlajide merged commit c190755 into main Feb 14, 2024
@BolajiOlajide BolajiOlajide deleted the bo/batch-changes-refresh-github-installs branch February 14, 2024 18:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants