redirect <reload>

I can't reproduce this, but then I can't reproduce the redirect at all, so this appears to be broken.

Actually I can reproduce this, the issue is that the redirect_path is being converted to String(), but the contents of that are nonsensical even if you have a redirect such as /node/1. I can however see that the redirect is called continuously.

Hi @aaron.ferris,

the issue happening here is the following:

On the configuration screen, the redirect time is being set as ["'time'"]. When we try to convert this value to a number, it returns NaN because it can’t properly interpret this format — the value is coming as a string with single quotes inside double quotes.

To fix this, we need to remove the single quotes (') from the value. In fact, this value doesn't even need to be a string at this point.

https://git.drupalcode.org/project/jstimer/-/blob/2.0.x/modules/jst_timer/jst_timer.module?ref_type=heads#L260

I'm leaving this fix for someone who's just getting started, as it's a great opportunity to understand how contributions work in Drupal.

I started working on it

Hello @charlliequadros

Thanks for that, very helpful and thanks @jonathandealmeida for taking a look.

I finished the fix, after the review, this can to go to the tests.

Thanks jonathandealmeida , ill pull this down and take a look.

Tested this with a redirect time of 10 seconds and a redirect of node/1 and it did seem to function as expected. As above, will leave this open in case for further testing.

Tested this with a redirect time of 22 seconds and a redirect of node/1 and it did seem to function as expected. See attached GIF

Thanks @dani volpe. Merging.

aaron.ferris → credited brandonlira → .

aaron.ferris → credited joaopauloscho → .

Adding credit for all those involved in Zoocha contrib initiative, including mentors.