feat (anrok integration): add pull taxes service that is used in async mode #2984
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Context
Currently Anrok calls are sync and performed during invoice generation
Description
The goal of this improvement is to make calls to Anrok in dedicated job so that we can implement throttling for rate limit issues.
This PR adds job and service that will be called in async mode. The main goal of the service is to fetch taxes and finalize the invoice. Service for retrying failed invoices is basically subset of the new service, so the new service will be also used there.
The next step is to adjust:
As part of adjustment in the places where taxes are fetched in the synchronous way, we will call
Invoice::ProviderTaxes::PullTaxesAndApplyJoband returnTaxUnknownError