Account created on 9 January 2023, over 1 year ago
#

Merge Requests

More

Recent comments

๐Ÿ‡ฎ๐Ÿ‡ณIndia sandipta

Adding patch for the fix.

๐Ÿ‡ฎ๐Ÿ‡ณIndia sandipta

The format_size() function exists in other files as well. They are :

webform_submission_export_import.module
WebformSubmissionForm.php
WebformAdminConfigElementsForm.php
WebformManagedFileBase.php
EmailWebformHandler.php
WebformElementManagedFileLimitTest.php.

๐Ÿ‡ฎ๐Ÿ‡ณIndia sandipta

sandipta โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia sandipta

Adding patch as per requirement

๐Ÿ‡ฎ๐Ÿ‡ณIndia sandipta

We can configure it in the following way:
Navigate to View Settings -> Advanced -> Exposed Form -> Exposed Form Style -> Settings
Increase the Delay for textfield autosubmit.

Please let me know if this solves your issue.

๐Ÿ‡ฎ๐Ÿ‡ณIndia sandipta

Adding Patch for additional requirement

๐Ÿ‡ฎ๐Ÿ‡ณIndia sandipta

Adding patch as per problem statement

๐Ÿ‡ฎ๐Ÿ‡ณIndia sandipta

sandipta โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia sandipta

Checked everything,two files remain where DI hasn't been implemented yet:

1. src/Plugin/GoogleAnalyticsCounterResultProcessor/UrlAliasResultProcessor.php
2. src/GoogleAnalyticsCounterHelper.php

I'll begin working on them soon.
@simonbaese

๐Ÿ‡ฎ๐Ÿ‡ณIndia sandipta

sandipta โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia sandipta

sandipta โ†’ made their first commit to this issueโ€™s fork.

โœจ | Prevnext | Route options
๐Ÿ‡ฎ๐Ÿ‡ณIndia sandipta

Adding patch as per requirement

๐Ÿ‡ฎ๐Ÿ‡ณIndia sandipta

Adding patch for phpcs error fixes.

๐Ÿ‡ฎ๐Ÿ‡ณIndia sandipta

sandipta โ†’ made their first commit to this issueโ€™s fork.

โœจ | Rules | rules.api.php
๐Ÿ‡ฎ๐Ÿ‡ณIndia sandipta

Hello everyone, i've added a merge request. Please review.

โœจ | Rules | rules.api.php
๐Ÿ‡ฎ๐Ÿ‡ณIndia sandipta

sandipta โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia sandipta

sandipta โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia sandipta

sandipta โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia sandipta

Hi @Adrianm6254, i am not able to reproduce the issue, but on basis of the error log message, i am adding this patch, can you please try this out once and check if the issue you are facing is persisting or not.
Thanks

๐Ÿ‡ฎ๐Ÿ‡ณIndia sandipta

sandipta โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia sandipta

sandipta โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia sandipta

sandipta โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia sandipta

sandipta โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia sandipta

Adding a patch, please check.

๐Ÿ‡ฎ๐Ÿ‡ณIndia sandipta

sandipta โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia sandipta

So the issue is with the Rules module, this module is having a hard dependency on it and since the rules module is not enabled it is throwing an error.

Also there is already a similar issue present fo this module, please check this out
https://www.drupal.org/project/user_email_verification/issues/3376165 ๐Ÿ› Fatal error when "rules" module is missing RTBC

And there's a patch available for this too, i tested it out and it's working fine.
https://www.drupal.org/files/issues/2023-07-21/user_email_verification-r... โ†’

I suggest applying the patch and checking if the issue still persists, if yes then we can come up with a solution.

@jsidigital

๐Ÿ‡ฎ๐Ÿ‡ณIndia sandipta

I checked it out and the issue persists.
The email verification block is not getting added.
Also i tried placing other blocks, but nothing was geting added/populated.
So for me the entire place-block feature is not working.
@jsidigital

๐Ÿ‡ฎ๐Ÿ‡ณIndia sandipta

Hi, I have tested the MR and it is working fine.

๐Ÿ‡ฎ๐Ÿ‡ณIndia sandipta

You can install devel entity update module and use drush entup command to fix the issue.

๐Ÿ‡ฎ๐Ÿ‡ณIndia sandipta

sandipta โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia sandipta

sandipta โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia sandipta

Can we use a different pattern here to avoid dependency on upstream dependencies. Please check out https://www.hashbangcode.com/article/drupal-9-extending-drupal-base-clas....

๐Ÿ‡ฎ๐Ÿ‡ณIndia sandipta

Thanks for the insights @admirlju.

๐Ÿ‡ฎ๐Ÿ‡ณIndia sandipta

Definitely, I'll test it out and will let you know. Thanks for the patch. @Vivek Panicker

๐Ÿ‡ฎ๐Ÿ‡ณIndia sandipta

Hi @akshaydalvi212. Your patch is failing some tests. Please check them out.

๐Ÿ‡ฎ๐Ÿ‡ณIndia sandipta

sandipta โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia sandipta

Sure. I will test it out and will let you know. Thanks for the help ๐Ÿ™‚

๐Ÿ‡ฎ๐Ÿ‡ณIndia sandipta
๐Ÿ‡ฎ๐Ÿ‡ณIndia sandipta

sandipta โ†’ created an issue.

๐Ÿ‡ฎ๐Ÿ‡ณIndia sandipta

sandipta โ†’ made their first commit to this issueโ€™s fork.

Production build 0.69.0 2024