Should probably not display unlock link for :time strategy

[marcio497]

A link on login page to unlock account leads to an error when :time is used as unlock strategy. I modified views/shared/_devise_links.erb to omit the link in such cases (I hope thats the correct place).

http://gist.github.com/365873

[carlosantoniodasilva]

It seems reasonable to not show the unlock link, but I think we should just show it if and only if you have enabled :email, and not if you don't enable :time. It's due to the possibility of having both :email and :time enabled.
What do you think?

[marcio497]

You are right, I completely forgot its possible to set :both.

[josevalim]

Can you please create a new patch then but using git format-patch or forking Devise and sending us the commit link?

[josevalim]

Do not show unlock link unless strategy is e-mail. Closed by 731f156.