- Issue created by @lfersoy
I've created this patch in case the proposed feature makes sense for anyone.
- πΈπ¦Saudi Arabia abdulaziz zaid Riyadh
Hi lfersoy,
Thank you so much for your excellent work on this. I believe your approach is correct, and everything is functioning well after testing. When you have time, could you kindly review the code and make sure it adheres to the standard coding practices?
I really appreciate your effort!
Hi Abdulaziz, thanks for your comment. I didnβt find any further improvements needed in the patch regarding standard coding practices. However, please let me know if there's anything else to improve, and Iβd be happy to make the changes.
- Status changed to Needs review
4 months ago 2:04am 11 December 2024 - πΈπ¦Saudi Arabia abdulaziz zaid Riyadh
Sorry We need to update the patch.
- Merge request !18Issue #3475043: Ensure users meet requirements before redirecting to the verification code form. β (Merged) created by lfersoy
Hi Abdulaziz, I created a fork of the issue and updated the code accordingly.
-
abdulaziz zaid β
committed 8311888a on 2.0.x authored by
lfersoy β
Issue #3475043: Fallback to default process when TFA requirements are...
-
abdulaziz zaid β
committed 8311888a on 2.0.x authored by
lfersoy β
- πΈπ¦Saudi Arabia abdulaziz zaid Riyadh
Thanks lfersoy
The changes have been successfully merged. Thank you for improving this module.
The previous version will be released soon.
Automatically closed - issue fixed for 2 weeks with no activity.