Delhi, India
Account created on 6 March 2013, almost 12 years ago
#

Merge Requests

Recent comments

๐Ÿ‡ฎ๐Ÿ‡ณIndia joshi.rohit100 Delhi, India

MR is merged in 2.x branch

๐Ÿ‡ฎ๐Ÿ‡ณIndia joshi.rohit100 Delhi, India

for the workaround, could be to implement hook_mail_alter() and replaced the generation of encoded header with D9 implementation in custom module.

๐Ÿ‡ฎ๐Ÿ‡ณIndia joshi.rohit100 Delhi, India

Thanks for the contribution!

๐Ÿ‡ฎ๐Ÿ‡ณIndia joshi.rohit100 Delhi, India

Thanks for the contribution

๐Ÿ‡ฎ๐Ÿ‡ณIndia joshi.rohit100 Delhi, India

Added the initial module for feedback.

Attached screenshots of the admin form and output

๐Ÿ‡ฎ๐Ÿ‡ณIndia joshi.rohit100 Delhi, India

This is already fixed in ๐Ÿ“Œ Fix linting warnings on README.MD Fixed and ๐Ÿ“Œ Drupal Coding Standards Issues | phpcs Fixed . pls check with latest dev version

๐Ÿ‡ฎ๐Ÿ‡ณIndia joshi.rohit100 Delhi, India

This is merged!

๐Ÿ‡ฎ๐Ÿ‡ณIndia joshi.rohit100 Delhi, India

This is merged!

๐Ÿ‡ฎ๐Ÿ‡ณIndia joshi.rohit100 Delhi, India

Fixed some coding standards

๐Ÿ‡ฎ๐Ÿ‡ณIndia joshi.rohit100 Delhi, India

Adding patch

๐Ÿ‡ฎ๐Ÿ‡ณIndia joshi.rohit100 Delhi, India

Merged! Thanks!

๐Ÿ‡ฎ๐Ÿ‡ณIndia joshi.rohit100 Delhi, India

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

๐Ÿ‡ฎ๐Ÿ‡ณIndia joshi.rohit100 Delhi, India

Merged! Thanks!

๐Ÿ‡ฎ๐Ÿ‡ณIndia joshi.rohit100 Delhi, India

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

๐Ÿ‡ฎ๐Ÿ‡ณIndia joshi.rohit100 Delhi, India

So test failed patch failed which looks correct. Thus marking this needs review Patch #4 can be used!

๐Ÿ‡ฎ๐Ÿ‡ณIndia joshi.rohit100 Delhi, India

This is just a failed test patch (No need to merged this)

๐Ÿ‡ฎ๐Ÿ‡ณIndia joshi.rohit100 Delhi, India

Added annotation on test class

๐Ÿ‡ฎ๐Ÿ‡ณIndia joshi.rohit100 Delhi, India

This is fixed!

๐Ÿ‡ฎ๐Ÿ‡ณIndia joshi.rohit100 Delhi, India

missed uploading patch

๐Ÿ‡ฎ๐Ÿ‡ณIndia joshi.rohit100 Delhi, India

Added patch

๐Ÿ‡ฎ๐Ÿ‡ณIndia joshi.rohit100 Delhi, India

This is fixed!

๐Ÿ‡ฎ๐Ÿ‡ณIndia joshi.rohit100 Delhi, India

This is now available in 3.1.1 release. Thanks for contributing.

๐Ÿ‡ฎ๐Ÿ‡ณIndia joshi.rohit100 Delhi, India

Merged and tagged new release 7.x-1.2

Thanks!

๐Ÿ‡ฎ๐Ÿ‡ณIndia joshi.rohit100 Delhi, India

Closing as not required

๐Ÿ‡ฎ๐Ÿ‡ณIndia joshi.rohit100 Delhi, India

Uploading a patch for failing test to validate

๐Ÿ‡ฎ๐Ÿ‡ณIndia joshi.rohit100 Delhi, India

Adding test case for ::isSprinklrFeatureEnabled() and ::isSprinklrEnabledForCurrentNode() helpers of the SprinklrHelper service

๐Ÿ‡ฎ๐Ÿ‡ณIndia joshi.rohit100 Delhi, India

I am also facing the same issue and not working even with providing the -c option for phpunit file.
Not sure if this is because of some. directory path issue.

Project setup is like below -

Root project directory with vendor folder
-- docroot directory
-- -- core directory

Production build 0.71.5 2024