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

Add auto_deploy_on field to enable auto deployment #225

Merged
merged 8 commits into from
Nov 19, 2021
Merged

Conversation

noahingh
Copy link
Member

No description provided.

@noahingh noahingh added the component: server This issue or pull request is related to the feature of the server-side label Nov 19, 2021
@noahingh noahingh added this to the v0.4 milestone Nov 19, 2021
@noahingh noahingh linked an issue Nov 19, 2021 that may be closed by this pull request
Comment on lines +38 to +40
// The 'owner_id' field is the ID who activated the repository.
field.Int64("owner_id").
Optional(),
Copy link
Member Author

Choose a reason for hiding this comment

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

Save the owner and use the token of the user when Gitploy deploys for an auto-deployment.

if err != nil {
return nil, fmt.Errorf("failed to create a webhook: %s", err)
}

r.WebhookID = hid
r, err = i.Activate(ctx, r)
r.OwnerID = u.ID
Copy link
Member Author

Choose a reason for hiding this comment

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

Set the owner when it activates the repository.

@noahingh noahingh added the documentation Pull requests improves or adds to documentation label Nov 19, 2021
@noahingh noahingh marked this pull request as ready for review November 19, 2021 12:37
@noahingh noahingh merged commit ba07ace into main Nov 19, 2021
@noahingh noahingh deleted the auto-deploy branch November 19, 2021 12:39
@noahingh noahingh added the schema-migration This pull request has a schema-migration label Nov 22, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
component: server This issue or pull request is related to the feature of the server-side documentation Pull requests improves or adds to documentation schema-migration This pull request has a schema-migration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Provide the auto deployment for the push event
1 participant