Should redirect_to Override One-Time-Offers?

[jaswrks]

A user writes...

Redirect to Previous Page. I noticed that will bypass any first-time-login page set in s2member. I hope you can improve upon this process. A user, just coming in to log in we would prefer going to account. Someone reading a blog post should be redirected back to the blog post I guess we have to accept one or the other. But my main concern is the bypassing of the first-time, fifth-time, etc logins.

[jaswrks]

@WebSharks/team Any thoughts on this?
Should redirect_to override everything? Including One-Time Offers Upon Login too?

[raamdev]

Should redirect_to override everything? Including One-Time Offers Upon Login too?

I would say yes. Here's why:

If I'm specifying a URL to redirect, I expect the user to get redirected straight there. I wouldn't want them to get interrupted by anything in between, even One-Time-Offers. In fact, when I read the "One-Time-Offers (Upon Login)" panel, it sounds to me like it applies only to those logins that will be arriving at the Login Welcome Page (i.e., without a custom redirect_to).

That said, I can see how this might get confusing for someone if they use the redirect_to and the One-Time-Offers... if users nearly always get to the login page by attempting to view protected content, as opposed to always logging in via the normal login box, then they'd never see the One-Time-Offers.

However, I still stand by my opinion that redirect_to should always override One-Time-Offers. A simple explanation of that fact should just be added to the One-Time-Offers option panel.

[jaswrks]

@raamdev I agree with you. The note in the Dashboard would be a good idea, thank you.

I'd love to find a way to make this better. I think the biggest issue is that the Pro Login Widget uses redirect_to quite extensively. So integrating that really causes some problems for One-Time Offers, since just about every login uses the redirect_to variable.

I'll give it some more thought. I was just thinking maybe there is a way for us to make an exception in some cases; where perhaps the redirect_to variable is being passed automatically by the Pro Login Widget, and it's just using a default value. Maybe another variable, like &redirect_to_automatic to signal this to the One-Time Offer system.

[raamdev]

Maybe another variable, like &redirect_to_automatic to signal this to the One-Time Offer system.

That sounds like a good idea to me.

[jaswrks]

Moving this to the "Future Release" milestone.

[jaswrks]

Assigning this to myself