@avpaderno Its @ajay-mallah who has made fixes of #13 and committed the changes.
and being the co maintainer of the module, I have equal contribution in this module development so I am commenting here.
@avpaderno I have added the suggested changes as per #13, please review.
Hii @avpaderno i have reached out to @cmlara on drupal slack for the discussion about the proposed resolution, will find a solution for this soon and update the module.
@cmlara since my implementation is not changing the id of the main form so the form alter hooks from other modules are not ignored. and
This also supports a more comprehensive approach
@cmlara I thought of overriding the core's user entity form directly, as the hook form alter was getting way lengthy and also to follow the OOPs concept properly i avoided the form alter API.
@apaderno can you please Review it again as its already 19th July.
Vishal Prasad โ created an issue.
Hi @arijit-acharya!
Thank you for this feature reqquest, It would be better if we could make the otp length configurable too, using the same Config Form
status was set to priority by mistake
@paraderojether Can we review and merge it on priority?
@Berdir I have tested the 3377195-9.patch by @penyaskito and it works good to me too,
i think we can create a MR and Merge it.
hello @VladimirAus I Tried to reproduce the error following the process you have mentioned, but i didn't get this error also the form is working fine to me, either we need more info about how to reproduce this error or there may be some issue with any other module in your peoject, causing this issue,
Closing it for now.
Hello @paraderojether I have made the requested changes in the Readme file,
i thought Readme file standards are not the Phpcs concern so i never added this extension for PHPCS check,
well i added this extension too and made the required changes. Can you please review it
Thanks!
@clarkssquared i have updated the issue summary as per suggestion,
please review
@abhishek_gupta1 I have Raised the MR fixing the PHPCS issues, can you please review it
MR Link
I will be solving this issue so assigning to myself.
Vishal Prasad โ made their first commit to this issueโs fork.
@apaderno I have made some changes for the translatable strings, can you please review it
Thanks regards!
@rohitsharma401 I have viewed the README file content given by @deepak5423 , it seems alright and the standards have been followed up to the mark according to me,
You can add this README.md file to your project.
@vishal.kadam i made the required changes.
@geoanders i have Raised a MR with the drush commands to export content (by id or by type)
please review it
Vishal Prasad โ changed the visibility of the branch 3323624-drush-command-by-vishal to hidden.
Vishal Prasad โ made their first commit to this issueโs fork.
@vishal.kadam I have made these changes, please review.
Hello @vishal.kadam I have made the required changes as per phpcs suggestions, you can now review the module.
Thank you,
Thank you @praveen rani for identifying the issue, its now fixed for the new release 1.0.3, you can verify it now.
Vishal Prasad โ created an issue.
Vishal Prasad โ created an issue.
Can you please be little more specific?
Vishal Prasad โ created an issue.
Vishal Prasad โ created an issue.
Hello there!
i have Raised the MR for upgraded version of this module that is compatible with D9 and D10.
Please Review and Merge this MR.
Vishal Prasad โ made their first commit to this issueโs fork.
Vishal Prasad โ created an issue.
Vishal Prasad โ made their first commit to this issueโs fork.
Vishal Prasad โ made their first commit to this issueโs fork.
Vishal Prasad โ made their first commit to this issueโs fork.
Hi i just made the changes to the fork, and modified the Readme.txt
to <h2>Readme.md</h2>
.
Vishal Prasad โ made their first commit to this issueโs fork.
I had the same error
in that case if you have abackup of the configuration then that will be great and importing the configuration using drush cim
should solve the error, but if u don't have the backup then just run composer update
and that should solve the issue.