Captcha is for just simple image and text based captcha verification , whereas recaptcha is select images and some other advanced type of verification .
Tested the patch , it works fine for me. Created a MR for same.
@trackleft2 The test cases looks good to me . RTBC ++
@fago the issue did not reproduced , i tried the above mentioned steps
Not able to reproduce the issue , is the issue with this module - https://www.drupal.org/project/recaptcha_v3 → ?
@neerajsingh i have opened this MR - https://git.drupalcode.org/issue/3542145-3543140/-/merge_requests/1/.
Please review and merge .
Hii @trackleft2 , can you please write PHPUnit tests for it , i don't have much idea about them
@trackleft2 All phpcs issues resolved .
jurgenhaas → credited sourabhsisodia_ → .
@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 .
sourabhsisodia_ → made their first commit to this issue’s fork.
@aaron.ferris , @charlliequadros idea looks good and I think we can proceed this way . I would love to work on this idea
sourabhsisodia_ → made their first commit to this issue’s fork.
sourabhsisodia_ → made their first commit to this issue’s fork.
@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.
working on it
sourabhsisodia_ → made their first commit to this issue’s fork.
sourabhsisodia_ → made their first commit to this issue’s fork.
@niharika.s your MR is failing the if conditions are not correct
@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 ??
sourabhsisodia_ → made their first commit to this issue’s fork.