Skip to content

Fix Devise deprecation warnings

username-removed-386624 requested to merge devise-deprecations into master

What does this MR do?

Updates omniauth provider paths to use the new Devise/Rails 5 format.

Are there points in the code the reviewer needs to double check?

That OAuth still works (from my tests it did).

Also I wanted to make sure that send is okay to use here, would we prefer public_send for any reason?

Why was this MR needed?

There were deprecation warnings from Devise 4.0 deprecating the omniauth authentication paths we were using.

What are the relevant issue numbers?

Fixes #18110 (closed).

Does this MR meet the acceptance criteria?

cc: @stanhu @DouweM

Merge request reports