Problem/Motivation

Intuit has announced an update to their API here:

https://app.dp.intuit.com/e/es?s=2332625&e=551&elqTrackId=efd74c1a1b7a40...

There is a recommendation to upgrade to v6.2.0 of the quickbooks v3 php sdk. This module currently uses 6.1.0.

The 6.2 version of the SDK has a requirement for guzzlehttp/guzzle v7.9.0, which conflicts with Drupal core versions below 10.4.

Proposed resolution

Determine if the upcoming update will have any impact on this module. If it is determined that there will be a negative impact, provide an updated version of this module that requires D10.4 and v6.2 for the SDK

Remaining tasks

  1. Determine if the upcoming update will have any impact on this module
  • No: No further action required
  • Yes: Provide an updated version of this module that requires D10.4 and v6.2 or the SDK

Issue fork quickbooks_api-3501785

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

cobblestone.consulting created an issue. See original summary.

cobblestone.consulting’s picture

cobblestone.consulting commented
Issue summary: View changes

alexandersluiter made their first commit to this issue’s fork.

alexandersluiter’s picture

alexandersluiter commented

I took this module over after the original maintainer deleted their account and it was maintained by 'anonymous'. I haven't had a lot of time for it, but I was able to give it a heavy lift some years ago. It's been on maintenance-only mode since. My company is working toward our Drupal 11 upgrade, so I will make a bit of time to lift this module to D11. I will push the SDK to 6.2+ and require a minimum of Drupal 10.4, which everyone should be on at this point anyway. I'll get an MR put together. There's a few other bugs in it that should be address while I'm at it.

alexandersluiter opened merge request !6

alexandersluiter opened merge request !7

cobblestone.consulting’s picture

cobblestone.consulting commented

Thanks @alexandersluiter, I appreciate your work on this!

alexandersluiter opened merge request !8

alexandersluiter closed merge request !7

alexandersluiter closed merge request !6

alexandersluiter’s picture

alexandersluiter commented
alexandersluiter’s picture

alexandersluiter commented
Assigned: Unassigned » alexandersluiter
Status: Active » Fixed

Merged into the 2.0.x branch.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.