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

Support HMRC API V2 #138

Merged
merged 2 commits into from
Dec 6, 2024
Merged

Support HMRC API V2 #138

merged 2 commits into from
Dec 6, 2024

Conversation

Skumring
Copy link
Contributor

Version 2 is the recommended version of this API. Version 1 will be removed in January 2025.

https://developer.service.hmrc.gov.uk/api-documentation/docs/api/service/vat-registered-companies-api/2.0

Resolves issue - #137

@yolk
Copy link
Owner

yolk commented Nov 25, 2024

First thanks for this welcome addition!

But it seems you are using features requiring Ruby 3.0 and Rails/ActiveSupport (deep_merge). valvat still supports Ruby 2.6 and running standalone (without activerecord).

Could you work on the failing specs? If it is not possible I'll could try to find some time.

@Skumring
Copy link
Contributor Author

Could you work on the failing specs? If it is not possible I'll could try to find some time.

Good evening, I added new commit with bugfix, please take a look and rerun the checks

@yolk yolk merged commit d6a1655 into yolk:master Dec 6, 2024
23 checks passed
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.

2 participants