๐Ÿ‡ฎ๐Ÿ‡ณIndia @Shaikh Sadab

Account created on 23 February 2023, almost 2 years ago
#

Merge Requests

Recent comments

๐Ÿ‡ฎ๐Ÿ‡ณIndia Shaikh Sadab

i have fixed the issue please review the patch.

๐Ÿ‡ฎ๐Ÿ‡ณIndia Shaikh Sadab

please review the patch.

๐Ÿ‡ฎ๐Ÿ‡ณIndia Shaikh Sadab

We have tested your Merge Request, and it's almost correct. However, when we go to the settings >> form page to enable the payment field, we encounter the following error.

Drupal\Component\Serialization\Exception\InvalidDataTypeException: Object support when dumping a YAML file has been disabled. in Drupal\Component\Serialization\Yaml::encode() (line 25 of C:\wamp64\www\drupal-contrib\core\lib\Drupal\Component\Serialization\Yaml.php).

We have fixed this issue in the latest release.

๐Ÿ‡ฎ๐Ÿ‡ณIndia Shaikh Sadab

We tested your patch, but when we attempted to enable the payment feature, we encountered the below error:

Drupal\Component\Serialization\Exception\InvalidDataTypeException: Object support when dumping a YAML file has been disabled. in Drupal\Component\Serialization\Yaml::encode() (line 25 of C:\wamp64\www\drupal-contrib\core\lib\Drupal\Component\Serialization\Yaml.php).

We have fixed this issue in the latest release.

๐Ÿ‡ฎ๐Ÿ‡ณIndia Shaikh Sadab

we have fixed this issue in the latest release please check.

๐Ÿ‡ฎ๐Ÿ‡ณIndia Shaikh Sadab

jQuery Once is deprecated in Drupal 10. Please refer to the updated documentation for more details. I have fixed the issue and submitted a patch for review.

document:- https://www.drupal.org/node/3158256 โ†’

๐Ÿ‡ฎ๐Ÿ‡ณIndia Shaikh Sadab

Weโ€™ve fixed all dependency injection issues in the latest release. Thank you for your feedback!

๐Ÿ‡ฎ๐Ÿ‡ณIndia Shaikh Sadab

Weโ€™ve added the hook_help() implementation in the latest release of the module. This should provide the necessary help text and documentation directly within the Drupal interface. Please update to the newest version to access this feature. Thank you for your feedback!

๐Ÿ‡ฎ๐Ÿ‡ณIndia Shaikh Sadab

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

๐Ÿ‡ฎ๐Ÿ‡ณIndia Shaikh Sadab

I have reviewed the patch and determined that there is no need to change the info file for version compatibility, as we did not find any issues. I have pushed the MR. Please review it.

๐Ÿ‡ฎ๐Ÿ‡ณIndia Shaikh Sadab

I have updated the Twitter links to point to X (x.com). Please review the patch.

๐Ÿ‡ฎ๐Ÿ‡ณIndia Shaikh Sadab

Hi,

We have fixed the construct error. However, after fixing it, we encountered another issue:

Error: Type of Drupal\same_page_preview\Form\SettingsForm::$typedConfigManager must be Drupal\Core\Config\TypedConfigManagerInterface (as in class Drupal\Core\Form\ConfigFormBase) in C:\wamp64\www\drupal-alpha\modules\contrib\same_page_preview\src\Form\SettingsForm.php on line 15

To resolve this error, we needed to use "typedConfigManager" since we are using the "ConfigFormBase". We have fixed the issue. Please review the patch.

Thank you!

๐Ÿ‡ฎ๐Ÿ‡ณIndia Shaikh Sadab

We have added the description and screenshots on the module page

๐Ÿ‡ฎ๐Ÿ‡ณIndia Shaikh Sadab

I was able to reproduce the issue. I have found a workaround for this issue. Please find the attached patch for the same.

Production build 0.71.5 2024