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.

Sign up for GitHub

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

Jump to bottom

feat: #81 add webhooks for invoices #106

Merged
merged 4 commits into from
Apr 26, 2022
Merged

feat: #81 add webhooks for invoices #106

merged 4 commits into from
Apr 26, 2022

Conversation

jdenquin
Copy link
Contributor

No description provided.

@jdenquin jdenquin added Feature Add a new feature to the app API Related to the public API Invoice Related to Invoices 🔔 Webhook Related to Webhooks labels Apr 26, 2022
@jdenquin jdenquin requested a review from vincent-pochet April 26, 2022 13:25
@jdenquin jdenquin self-assigned this Apr 26, 2022
@jdenquin jdenquin changed the title 81 add webhooks for invoices feat: #81 add webhooks for invoices Apr 26, 2022
@jdenquin jdenquin linked an issue Apr 26, 2022 that may be closed by this pull request
Add webhooks for invoices #81
Closed
vincent-pochet
vincent-pochet approved these changes Apr 26, 2022
View reviewed changes
Copy link
Collaborator

@vincent-pochet vincent-pochet left a comment

Choose a reason for hiding this comment

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

Looks good 🚀!

app/jobs/send_webhook_job.rb Outdated Show resolved Hide resolved
app/serializers/v1/invoice_serializer.rb Outdated Show resolved Hide resolved
app/serializers/webhook_serializer.rb Outdated Show resolved Hide resolved
@jdenquin jdenquin force-pushed the 81-add-webhooks-for-invoices branch from d87161d to d2b54dc Compare April 26, 2022 14:20
@jdenquin jdenquin merged commit a851fd8 into main Apr 26, 2022
@jdenquin jdenquin deleted the 81-add-webhooks-for-invoices branch April 26, 2022 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vincent-pochet vincent-pochet vincent-pochet approved these changes

Assignees

@jdenquin jdenquin

Labels
API Related to the public API 🔔 Webhook Related to Webhooks Feature Add a new feature to the app Invoice Related to Invoices
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add webhooks for invoices
2 participants
@jdenquin @vincent-pochet