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

feat (accounting-integrations): add logic for calling nango sync endpoint #1869

Merged
merged 3 commits into from
Apr 16, 2024

Conversation

lovrocolic
Copy link
Collaborator

Context

Currently Lago does not support accounting integrations

Description

This PR adds base service for communicating with Lago's accounting integrations aggregator - Nango.

Also, the sync job is added. Lago needs to call Nango sync endpoint upon integration object creation. Sync is needed for Nango so that they can pull data from accounting provider.
Nango's enpoint structure will be the same for all other providers.

Copy link
Contributor

@ivannovosad ivannovosad 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/services/integrations/aggregator/base_service.rb Outdated Show resolved Hide resolved
@lovrocolic lovrocolic merged commit f79e82d into main Apr 16, 2024
6 checks passed
@lovrocolic lovrocolic deleted the ftr-add-nango-sync branch April 16, 2024 08:15
drejc pushed a commit to fliqa-io/lago-api that referenced this pull request May 15, 2024
…oint (getlago#1869)

## Context

Currently Lago does not support accounting integrations

## Description

This PR adds base service for communicating with Lago's accounting
integrations aggregator - Nango.

Also, the sync job is added. Lago needs to call Nango sync endpoint upon
integration object creation. Sync is needed for Nango so that they can
pull data from accounting provider.
Nango's enpoint structure will be the same for all other providers.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants