Rajasthan, India
Account created on 9 December 2019, about 5 years ago
#

Merge Requests

More

Recent comments

🇮🇳India rakesh.regar Rajasthan, India

rakesh.regar made their first commit to this issue’s fork.

🇮🇳India rakesh.regar Rajasthan, India

rakesh.regar made their first commit to this issue’s fork.

🇮🇳India rakesh.regar Rajasthan, India

rakesh.regar made their first commit to this issue’s fork.

🇮🇳India rakesh.regar Rajasthan, India

rakesh.regar made their first commit to this issue’s fork.

🇮🇳India rakesh.regar Rajasthan, India

Raised Merge Request with the #15 patch.

🇮🇳India rakesh.regar Rajasthan, India

rakesh.regar made their first commit to this issue’s fork.

🇮🇳India rakesh.regar Rajasthan, India

Raised Merge Request.

🇮🇳India rakesh.regar Rajasthan, India

rakesh.regar made their first commit to this issue’s fork.

🇮🇳India rakesh.regar Rajasthan, India

Hey @apaderno

I have made some changes in PR and had some confution about the comment, please check it.
Thanks.

🇮🇳India rakesh.regar Rajasthan, India

MR raised, moving to needs review.

🇮🇳India rakesh.regar Rajasthan, India

rakesh.regar made their first commit to this issue’s fork.

🇮🇳India rakesh.regar Rajasthan, India

This issue seems to be already fixed on the latest dev branch, I have verified attaching screenshot for reference.

🇮🇳India rakesh.regar Rajasthan, India

MR raised updating status to needs review.

🇮🇳India rakesh.regar Rajasthan, India

rakesh.regar made their first commit to this issue’s fork.

🇮🇳India rakesh.regar Rajasthan, India

This issue is already fixed in 2.x branch so moving it to fixed

🇮🇳India rakesh.regar Rajasthan, India

Added a link to create a one time login link from profile page. moving to needs review.

🇮🇳India rakesh.regar Rajasthan, India

rakesh.regar made their first commit to this issue’s fork.

🇮🇳India rakesh.regar Rajasthan, India

Feature is developed and MR raised, please review. moving to Needs review.

🇮🇳India rakesh.regar Rajasthan, India

I tried to add feature to set different validation time for each user and it seems it will take alot time as we need to override the default reset password functions

so I'll add a link to a form that will allow to customize valid time for all users.

🇮🇳India rakesh.regar Rajasthan, India

I'm working on this.

I'll add a confirmation popup where the default time will be there and if you wish to change the time you can generate link.

🇮🇳India rakesh.regar Rajasthan, India

I have tested these changes and it works fine.

But this change disable cache for user account block, for people using page as destination we can use this but for people who are not using as detination, it is not recommanded.

Disabling cache will reduce performance, better solution would be having some kind of check using which it can be determind if a user have set redirection to the current page only then we should stop caching this block.

🇮🇳India rakesh.regar Rajasthan, India

This is a duplicate issue of - https://www.drupal.org/project/login_destination/issues/3231726 🐛 Redirect destination not working Active

please check before adding a new issue in issue queue.

🇮🇳India rakesh.regar Rajasthan, India

MR raised please review.

🇮🇳India rakesh.regar Rajasthan, India

MR updated please Review.

🇮🇳India rakesh.regar Rajasthan, India

rakesh.regar made their first commit to this issue’s fork.

🇮🇳India rakesh.regar Rajasthan, India

MR rasied moving to needs review.

🇮🇳India rakesh.regar Rajasthan, India

MR Raised moving it to Needs Review.

🇮🇳India rakesh.regar Rajasthan, India

Task is complete and MR is rasied moving issue to Needs review.

🇮🇳India rakesh.regar Rajasthan, India

MR raised moving to Needs Review.

🇮🇳India rakesh.regar Rajasthan, India

Completed the task moving issue to needs review.

🇮🇳India rakesh.regar Rajasthan, India

Raised MR, moving issue to needs review.

🇮🇳India rakesh.regar Rajasthan, India

Added More Bulk operation options
- Activate Selected
- Deactivate Selected
- Delete Selected
- Move selected to Header
- Move Selected to Body
- Move selected to Footer

updating status to needs review

🇮🇳India rakesh.regar Rajasthan, India

Raised MR, updating status to Needs Review.

🇮🇳India rakesh.regar Rajasthan, India

I have raised a MR, updating status to Needs Review

🇮🇳India rakesh.regar Rajasthan, India

Merge Request is raised. so, updating status from active to needs review.

🇮🇳India rakesh.regar Rajasthan, India

I'm working on this issue.

🇮🇳India rakesh.regar Rajasthan, India

Moving it to RTBC as Greg Boggs has verified changes.

🇮🇳India rakesh.regar Rajasthan, India

Hi @Greg Boggs

I have opened a MR for the same.

🇮🇳India rakesh.regar Rajasthan, India

please use this new patch.
Instead of using mb_strtolower function 2 times, I have removed it from custom path so it will increase performance while resolving the issue.

🇮🇳India rakesh.regar Rajasthan, India

Hi @pankaj1390

I found the issue, created a patch for the same.

🇮🇳India rakesh.regar Rajasthan, India

rakesh.regar made their first commit to this issue’s fork.

🇮🇳India rakesh.regar Rajasthan, India

This module didn't required much changes for to be working on Drupal 9 and 10, I have created a patch that can fix the issue.

🇮🇳India rakesh.regar Rajasthan, India

@pankaj1390
I checked this and was not able to reproduce this issue, please add steps to reproduce

Find attached screenshot of breadcrumbs working fine

🇮🇳India rakesh.regar Rajasthan, India

moving from active to needs review

🇮🇳India rakesh.regar Rajasthan, India

Hi Neslee Canil Pinto

The stable build does not contain the latest code from the default branch
because of that this issue is appearing

Please update the stable build code.

Hi RgnYLDZ
I'm adding a patch for this error.

Production build 0.71.5 2024