-
Notifications
You must be signed in to change notification settings - Fork 67
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
URI.escape is deprecated #36
Comments
It broke in ruby 3.0, I will be forced to fork this project if it is dead and not accepting PRs |
|
Fixed by #35 |
you can simple use def user_avatar user
"https://www.gravatar.com/avatar/#{Digest::MD5.hexdigest(user.email.downcase)}?size=150"
end |
@msroot perfect, thanks! We had to add a def user_avatar user
"https://www.gravatar.com/avatar/#{Digest::MD5.hexdigest(user.email.downcase)}?default=identicon&size=150"
end |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Running on Ruby 2.7, I get this warning a lot:
Everything still seems to work fine, but I'm afraid it might break on a newer version.
The text was updated successfully, but these errors were encountered: