Skip to content
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

Trigger manually sync/full sync #379

Open
martinm82 opened this issue Jan 12, 2023 · 3 comments
Open

Trigger manually sync/full sync #379

martinm82 opened this issue Jan 12, 2023 · 3 comments
Labels
enhancement New feature or request

Comments

@martinm82
Copy link
Contributor

Prerequisites:

  • Is the functionality available in the GitHub UI? If so, please provide a link to information about the feature.

New Feature

Imagine you have the situation where the safe-settings app did not configure a repository properly, because for whatever reason, e.g., webhook not send to the app, an application error, etc..

Wouldn't it be great if there would be at least the possibility to trigger manually a sync or full sync through a dedicated endpoint?

@martinm82 martinm82 added the enhancement New feature or request label Jan 12, 2023
@decyjphr
Copy link
Collaborator

This was solved for lambda by exporting syncInstallation method in the index.js #313

Alternatively, we can create an HTTP endpoint in the app that would do the same.

@anderssonjohan
Copy link
Contributor

@martinm82 I'm thinking I will try to set up a GitHub Actions workflow using cron + workflow dispatch (manual run) to call the lambda function.
I have just started using this app but find it hard to get feedback on what happened for the real run after PRs are merged to the main branch of the admin repo. It would be great to be able to re-run the changes on main and get feedback from the lambda back as a commit status. The dry run is enough for PRs but is not good enough for main IMHO.

@pydolan
Copy link
Contributor

pydolan commented Feb 28, 2024

As noted on #378, I have a prototype runner that I'm using for running safe-settings via GHA. If we like this approach of calling syncInstallation via a separate script, I can open a PR (was just delaying since I forked off my changes in #588) update: PR opened in #604

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants