Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update Gem to Address Moneybird API Warning Responses #85

Merged

Conversation

leandrost
Copy link
Contributor

This pull request updates the gem to address Moneybird's recent API warnings encountered during interactions.

CleanShot 2024-05-07 at 12 43 19@2x

The resource classes TaxRate, Contact, SalesInvoice, and Invoice::Details were changed to add the missing attributes following the latest Moneybird API specifications to prevent warning messages.

Below are the attributes added:

  • TaxRate: country
  • Contact: archived, contact_people, moneybird_payments_mandate
  • SalesInvoice: contact_person, contact_person_id, next_reminder, original_estimate_id, public_view_code_expires_at, reminder_count, short_payment_reference
  • Invoice::Details: mandatory_tax_text

Additional Notes

Changes should be backward compatible with previous versions of the gem (only additions, no attributes was removed)
All modifications have been tested locally to ensure functionality with the updated Moneybird API.
Further testing from other integrations is recommended to validate changes before deployment.

Missing resource attributes added:
  - archived
  - contact_people
  - moneybird_payments_mandate
Missing resource attribute added: mandatory_tax_text
Missing resource attributes added:
  - contact_person
  - contact_person_id
  - next_reminder
  - original_estimate_id
  - public_view_code_expires_at
  - reminder_count
  - short_payment_reference
Missing resource attribute added: country
@leandrost leandrost marked this pull request as ready for review May 7, 2024 18:48
@maartenvanvliet
Copy link
Owner

Thank you, ideally the test fixtures should also be updated but that's not necessary right now.

@maartenvanvliet maartenvanvliet merged commit a251fa2 into maartenvanvliet:master May 8, 2024
1 of 3 checks passed
@leandrost leandrost deleted the add-missing-attributes branch May 8, 2024 13:59
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants