Account created on 27 October 2023, over 1 year ago
  • Associate Software Engineer at Innoraft 
#

Merge Requests

More

Recent comments

@socialnicheguru if we are using this "Set timezone on login only if it is not yet set (recommended)" option then we have to do the below steps -
Go to Admin → Configuration → Regional settings
Set "Default time zone" to "empty timezone" .

I have created MR as in the error message we are getting a link that redirects to wrong settings page .

@aaron.ferris , @charlliequadros idea looks good and I think we can proceed this way . I would love to work on this idea

@chsdaiguil the first issue for empty style attribute is reproduced and i have solved it , but the second issue of duplicate id is not reproduced at all . I have tried placing multiple disclosure menu on the same page but its not reproduced at all .

@asawari from the image it seemed like you tested the MR on issue page , not on local setup.

@zenimagine If everything looks good then please move the issue to RTBC state

This is the last Commit . And it has bring changes in css only . was the patch working previously ??

@tim-diels Resolved merge conflicts and test cases.

@zenimagine Resolved the issue as mentioned above , please review now.

hii @ady1503, if the patch works fine then please move the issue to " Review and Tested By Community " state and mention the maintainer to review and merge the MR . So that this issue get fixed in the dev version also.

@ady1503 Attaching a screenshot . Is it the correct configuration for the issue ??

@kongkx while installing the module i am getting this error - Error: Class "DeviceDetector\ClientHints" not found in Drupal\visitors\Service\DeviceService->getDeviceDetector()

@l-laziz issue did not reproduced at all , attaching screenshot for refernce,

@blacksnipe thanks for approving the MR , since the MR is merged then we have to move the issue to fixed state instead of RTBC . And also check the credit and commiting part present below in this issue page.

@siramsay I have reviewed the changes and the changes looks good. Although I have fixed some code formatting issue in your commit and after pushing code to a branch we need to create a MR into the branch we need to implement these changes . So raised a MR.
Attaching screenshot for reference . RTBC ++.

Production build 0.71.5 2024