Account created on 21 August 2015, about 9 years ago
#

Recent comments

πŸ‡ΊπŸ‡ΈUnited States wstocker

Second the comment in #11

I have a fresh install of D10 and I had to apply this patch when using combinations of configuration or the whole page would crash.
Seems like it's needed.

πŸ‡ΊπŸ‡ΈUnited States wstocker

Adding patch file to fix this issue.

πŸ‡ΊπŸ‡ΈUnited States wstocker

Upon further inspection this is a dupe of this issue https://www.drupal.org/project/token/issues/3397358 πŸ› 3218969 breaks some AJAX forms Active
Closing this out.

πŸ‡ΊπŸ‡ΈUnited States wstocker

Refactored thunder_admin-3354603-14.patch addressing error where a jQuery object was incorrectly passed to the Drupal once utility, which expects raw DOM elements.

πŸ‡ΊπŸ‡ΈUnited States wstocker

Patch 14 and D10.2 gives me this error where paragraph fields are used.

drupal.js?v=10.2.4:64 Uncaught TypeError: The element must be an instance of Element
    at u (once.js:122:11)
    at i (once.js:156:25)
    at a (once.js:280:5)
    at Object.attach (paragraphs.js?s9xsr0:16:7)
    at drupal.js?v=10.2.4:166:24

I don't think this is properly handling elements returned by once.

πŸ‡ΊπŸ‡ΈUnited States wstocker

Patch 14 is causing errors when deleting on D10 because of a missing access check on the query. As Berdir mentions this patch is obsolete by adding the purger queue.

πŸ‡ΊπŸ‡ΈUnited States wstocker

Patch #28 is incompatible with D10 as it uses deprecated and removed file_create_url().
Refactored so it is compatible with D10 and 3.x.

πŸ‡ΊπŸ‡ΈUnited States wstocker

Re-rolled with 2.x.

To generated a random number, I combined current timestamp (time()) with a random number (rand()), further reducing the likelihood of duplicates.

πŸ‡ΊπŸ‡ΈUnited States wstocker

Adding needs reroll tag. I will create a patch that will address sergei_brill's concern.

πŸ‡ΊπŸ‡ΈUnited States wstocker

Patch 26 applied, but to Damiens' point there were PHP errors surrounding the new field name in #20.

We had been using patch #13 for several years which has field_image instead of field_media_image.

I think making that configurable is a good idea, but for now, I re-rolled and created patch #28 using patch #13 to start and the errors disappeared.

Patch applies clean to 8.x-1.x.

πŸ‡ΊπŸ‡ΈUnited States wstocker

@sakthi_dev Whoops apologies wrong file. Here is the updated one.

πŸ‡ΊπŸ‡ΈUnited States wstocker

Patch re-rolled and updated with Drupal core 9.5.x.

πŸ‡ΊπŸ‡ΈUnited States wstocker

This might be related to this issue: https://www.drupal.org/project/stable/issues/3346133 πŸ“Œ Update off-canvas overrides after core refactoring Needs review

πŸ‡ΊπŸ‡ΈUnited States wstocker

I'm on Drupal 9.5.10 and Vimeo thumbnails for private videos have stopped working.

πŸ‡ΊπŸ‡ΈUnited States wstocker

Second that on Orkut Murat YΔ±lmaz' comment :)

πŸ‡ΊπŸ‡ΈUnited States wstocker

Simply renaming the file in patch #16 worked for me on 10.1.1

πŸ‡ΊπŸ‡ΈUnited States wstocker

Thanks! Yes. We have a work around for now, but good to know for the future.

πŸ‡ΊπŸ‡ΈUnited States wstocker

Wanted to throw another update here. In order to get around this issue we are having to re-build the tracker locally where we can set the PHP memory limit to -1 and upload the the entire database to each environment on the host where tracking is hitting the memory limit.

If tracking gets stuck in this memory limit there is essentially no way to unstick it. Rebuilding tracking info entirely hits the memory limit and then track now starts hitting the memory limit immediately. This also blocks cron from running since it's trying to run the same task.

πŸ‡ΊπŸ‡ΈUnited States wstocker

Uploading patch file works with Drupal 9 PHP 8.

πŸ‡ΊπŸ‡ΈUnited States wstocker

Marking this closed dupe of https://www.drupal.org/project/address_autocomplete/issues/3311019 πŸ› Module configuration issues Needs review

πŸ‡ΊπŸ‡ΈUnited States wstocker

Changing status to "Reviewed & testing by the community."

πŸ‡ΊπŸ‡ΈUnited States wstocker

MR!68 Looks good to me. Tested with Drupal 9 PHP 8.

πŸ‡ΊπŸ‡ΈUnited States wstocker

Additionally we are setting the API key in settings.php:

$config['address_autocomplete.settings']['google_maps']['api_key']

If I remove that line the page load ok, but would like the ability to set this in a private settings file.

πŸ‡ΊπŸ‡ΈUnited States wstocker

Turns out I created the app in the wrong place on the Hubspot side. Once I created the app under "Private Apps" I was able to connect successfully with the access token it generated.

Changing the status to "Reviews and tested by the community".

πŸ‡ΊπŸ‡ΈUnited States wstocker

Changing status to "Needs work".

πŸ‡ΊπŸ‡ΈUnited States wstocker

System settings:
PHP 8.0.24
Drupal core 9.5.3
ddev version v1.21.3

Installed patch and cannot connect. Message is "Cannot connect to hubspot API."

Steps to reproduce:

  • Require dev constraint in composer.json "drupal/hubspot": "dev-3308782-private-app as 3308782"
  • Include the repository in composer.json: "https://git.drupalcode.org/issue/hubspot-3308782.git"
  • composer update to install patch
  • Run drush updbFires hook obsolete API settings. Implements hook_update_N().
  • After obtaining the access token from hubspot API paste in access token field and save

Response error message:

SevenShores\Hubspot\Exceptions\BadRequest: Client error: `GET https://api.hubapi.com/integrations/v1/me` resulted in a `401 Unauthorized` response: {"status":"error","message":"Authentication credentials not found. This API supports both API Key and OAuth 2.0 authenti (truncated...) in SevenShores\Hubspot\Exceptions\HubspotException::create() (line 24 of /var/www/html/vendor/hubspot/hubspot-php/src/Exceptions/HubspotException.php).

Production build 0.71.5 2024