Check reset password token mutation #10

mcelicalderon · 2019-09-03T05:08:08Z

No description provided.

00dav00 · 2019-09-09T11:53:01Z

app/graphql/graphql_devise/mutations/base.rb

@@ -31,6 +31,11 @@ def controller
        context[:controller]
      end

+      def set_auth_headers(resource)


set_auth_headers is already present in line 63 of this fiel

00dav00 · 2019-09-09T12:01:03Z

app/graphql/graphql_devise/mutations/check_password_token.rb

+      private
+
+      def client_and_token(token)
+        if Gem::Version.new(DeviseTokenAuth::VERSION) <= Gem::Version.new('1.1.0')


Perhaps we can create a method for this check, It is also used in client method on the base mutation

mcelicalderon force-pushed the check-reset-password-token branch 2 times, most recently from eb75708 to c6eb381 Compare September 9, 2019 01:16

mcelicalderon marked this pull request as ready for review September 9, 2019 01:16

Check reset password token mutation

22c8b5b

mcelicalderon force-pushed the check-reset-password-token branch from c6eb381 to 22c8b5b Compare September 9, 2019 01:24

00dav00 requested changes Sep 9, 2019

View reviewed changes

Remove duplicate set_auth_headers definition

c1c6183

00dav00 approved these changes Sep 9, 2019

View reviewed changes

mcelicalderon merged commit 457c093 into master Sep 9, 2019

mcelicalderon deleted the check-reset-password-token branch September 9, 2019 16:59

mcelicalderon added the enhancement New feature or request label Oct 30, 2019

Provide feedback