Skip to content

Conversation

@jkowens
Copy link
Contributor

@jkowens jkowens commented Jan 16, 2021

Hoping this helps the Devise team as they work on releasing an update to support OmniAuth 2.0.

See: heartcombo/devise#5326 (comment)

@jkowens
Copy link
Contributor Author

jkowens commented Jan 17, 2021

@BobbyMcWho would you be able to review this PR? Sorry I know you’ve been busy 😬

@BobbyMcWho
Copy link
Contributor

@jkowens sure, sorry I don't get notifications in this repo on activity. I'll take a look today as soon as I can, and I'll figure out who I need to herd to get this released on rubygems.

@BobbyMcWho
Copy link
Contributor

It looks like @mbleigh is the owner on rubygems

@BobbyMcWho
Copy link
Contributor

@jkowens Just need the version file update committed since it looks like the Gemfile.lock has this gem bumped to 2.0.0?

@jkowens
Copy link
Contributor Author

jkowens commented Jan 17, 2021

@BobbyMcWho I think the reason for the version change in Gemfile.lock is because the gem was released as 2.0, but I don't think the Gemfile.lock had been updated.

@BobbyMcWho
Copy link
Contributor

Ah you're correct, the main branch has v2.0.0 but it's never been released on rubygems.

@carlosantoniodasilva
Copy link

Thank you both for your work on this! ❤️

@BobbyMcWho BobbyMcWho merged commit 8ee8ef3 into ruby-openid:master Jan 19, 2021
@BobbyMcWho
Copy link
Contributor

I'll release this as 2.0.0 later, I need to do some history digging and see what has been merged since the last rubygems release was cut.

@BobbyMcWho
Copy link
Contributor

@carlosantoniodasilva @jkowens I released it as v2.0.1 just to be safe.

@carlosantoniodasilva
Copy link

Perfect, thanks. I'll look into the devise updates next, hopefully this week still.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants