url redirect

sunnysha

i dont know if this is the proper way to ask the question.
but if i get a link to a particular discussion like
http://vanillaforums.org/discussion/16662/top-posters-for-the-month

and i am signed out from the forum.
when i click on add comment, the page i get to after login is the page i have the link for.
but if i sign in through the sign in button on left i am redirected to the discussions homepage.
is vanilla supposed to be so?
a user might have trouble getting to that particular page if after signing in through the sign in button from the left, he is redirected to all discussions page.

Linc

A good point. I filed a bug ticket. https://github.com/vanillaforums/Garden/issues/1049

luc

@Lincoln: doesn't https://github.com/vanillaforums/Garden/commit/6dbcc948f5c1fb3c00b379d93c899243f85d17f2 fix this?

Linc

A good point. I filed a bug ticket. https://github.com/vanillaforums/Garden/issues/1049

luc

@Lincoln: doesn't https://github.com/vanillaforums/Garden/commit/6dbcc948f5c1fb3c00b379d93c899243f85d17f2 fix this?

Linc

It does, I did not see that

batters

not help, in my default.master.php file not exist this string $this->Menu->AddLink('Entry', T('Sign In'), SignInUrl(), FALSE, array('class' => 'NonTab'), $Attribs);
exist different string $this->Menu->AddLink('Entry', T('Sign In'), Gdn::Authenticator()->SignInUrl(), FALSE, array('class' => 'NonTab'), $Attribs);

if im change 36 string, like in tip then appears error, and site dont work

luc

@batters, there was some more changes in the upcoming 2.0.18 that you don't have yet.
But it's a good indication that should work as you asked when it's released.
Maybe you could try a new install (using latest unstable: https://github.com/vanillaforums/Garden/zipball/unstable) to see if it fixes it for you.