the patch in #22 do not work with 8.x-7.0-alpha5
here is patch
dastan56 → created an issue.
dastan56 → created an issue.
this issue patch seems to be the cause https://www.drupal.org/project/google_tag/issues/3452712 🐛 Possibly script loading/placement issue Fixed
they load all the libraries with header: true , the gtag.ajax depends on core/drupal.ajax which breaks the ajax components on the page if loaded with header: true
thanks for the quick response, please give credit for the patch, thanks
patch attached.
dastan56 → created an issue.
i have created a patch from the diff in #13
thanks, i have attached the patch
i further worked on it, and changed this code in service ScriptService.php on line 433 from this:
$script .= "
document.addEventListener('readystatechange', () => {
// The first readyStateChange will be : interactive
// The second readyStateChange will be : complete
if (document.readyState === 'complete') {
tracker();
}
});
";
to this:
$script .= "
window.addEventListener('load', function() {
tracker();
});
seems to fix the issue, now the md_slider loads in the safari, and the visit is also counted, but it causes another, issue the counter is cached for anonymous users, so although the visit is counted but the number is updated only when the cache is cleared, is this ok? should i create a patch for it please let me know
these are the versions i use
Drupal Version: 10.3.2
PHP Version 8.2.22
Visitors: 8.x-2.19
md_slider: 1.5.4
yes sorry, i did further debugging and found that the tracking script was instead causing the issue so when i checked Disable tracking under advanced settings then the slider loaded in safari, basically the script prevents the $(window).on('load', function(){ from executing in the md_slider init-md-slider.js file and hence the slider is not loaded
dastan56 → created an issue.
hi, i can develop such a module for you, please contact me via my account contact form and we can discuss further, thanks
ok, thanks again, as i said for now using this code has resolved this issue for me:
\Drupal::service('cache.default')->delete('honeypot_protected_forms');
i will keep an eye on it and see if the form freezes again.
Thank you for your response, but there is one thing you did not address, which is that the error contains this:
ON DUPLICATE KEY UPDATE "cid" =
as you see the error happens, because when it tries to add honeypot_protected_forms to the cache_default table, it gets error that the key already exists, and this is why i delete it so that this duplication error donot happen, thanks again.
dastan56 → created an issue.
while above patch fixed the permission issue, now there is a new error for anonymous users:
TypeError: Drupal\openai_assistants\Entity\OpenaiThread::getAssistantId(): Return value must be of type string, null returned in Drupal\openai_assistants\Entity\OpenaiThread->getAssistantId() (line 235 of /web/modules/contrib/openai_assistants/src/Entity/OpenaiThread.php).
i did some investigation, and found the reason is because the assistant_id which is stored using tempstore.private is only stored for anonymous users when the cache is cleared, so if cache is not cleared then the assistant_id is not stored which then gives error, so following the instruction here https://www.drupal.org/forum/support/module-development-and-code-questio... → i added this code to both DialogForm and OpenaiChatForm
$form['#cache'] = ['max-age' => 0];
this resolved the issue for anonymous users, i have attached the patch which includes both the permission patch above and the cache changes.
i am attaching the patch which adds this permission
dastan56 → created an issue.
dastan56 → created an issue.
Hi, any update on this? till can we expect a release with the OAuth functionality, thanks
The attached patch fixes the .info.yml file
the patch in #4 was not getting applied, i am attaching a working patch.
ok, sure, we can close this issue then, as it seems done.
here is the patch
dastan56 → created an issue.
hi, thanks for the patch, there is a new issue that i noticed, its when i have unchecked multiple submission checkbox, then if i do the submission first time, it works good, it fires only once, but when i try to do submission second time then the subscriber is not fired at all, please check it, thanks.
dastan56 → created an issue.
if you are getting still issue in applying the patch #2 then following this article: https://chromatichq.com/insights/patching-info-files-composer/
i used this command and the patch applied cleanly:
composer require 'drupal/ajax_add_to_cart:^2.0' --prefer-source
dastan56 → created an issue.
hi, i am taking about the allowing us to set fastIframe to false in configuration page, when this property is set to false, it delays the loading graphic removal and onComplete event until iframe's content has completely loaded. And this is exactly what i want, the loading graphic to show until the iframe is completely loaded as my images are a little too big and takes some time to load, hope that makes sense
if you are getting still issue in applying the patch #22 then following this article: https://chromatichq.com/insights/patching-info-files-composer/
i used this command and the patch applied cleanly:
composer require 'drupal/bootstrap_styles:^1.1' --prefer-source
hi, the patch #11 was not working with 1.1.0, so i created this new patch, it removes media_library_form_element in both info.yml and composer.json file and replaces it with layout_builder_iframe_modal
dastan56 → created an issue.
@Spleshka hi, i am trying to override one of the anu lms react component in my custom module react, but everytime i try to import using following code:
import ParagraphsWithQuiz from '@anu/components/ParagraphsWithQuiz.js';
i get the following error, when i run npm run build:
Module not found: Error: Can't resolve '@anu/components/ParagraphsWithQuiz.js'
i see in the readme file you have written TODO. Untill you update the documentation, can you give me a quick idea here on how to get it done so that i can continue with my work, thanks
I figured it out myself, i did my own custom coding, this github repository provided grest help: