use only integers for org ids

Actually, using "native" integers works too, and seems simpler.

use only integers for org ids
Actually, using "native" integers works too, and seems simpler.
Jérémy Lecour
1 parent 54fcbb97
Showing 1 changed file with 2 additions and 2 deletions Show diff stats
app/controllers/users/omniauth_callbacks_controller.rb
@@ -8,8 +8,8 @@ class Users::OmniauthCallbacksController &lt; Devise::OmniauthCallbacksController
       # See if they are a member of the organization that we have access for
       # If they are, automatically create an account
       client = Octokit::Client.new(access_token: github_token)
-      org_ids = client.organizations.map { |org| org.id.to_s }
-      if org_ids.include?(github_org_id.to_s)
+      org_ids = client.organizations.map { |org| org.id }
+      if org_ids.include?(github_org_id)
         github_user = User.create(name: env["omniauth.auth"].extra.raw_info.name, email: env["omniauth.auth"].extra.raw_info.email)
       end
     end
...	...	@@ -8,8 +8,8 @@ class Users::OmniauthCallbacksController < Devise::OmniauthCallbacksController
8	8	# See if they are a member of the organization that we have access for
9	9	# If they are, automatically create an account
10	10	client = Octokit::Client.new(access_token: github_token)
11		- org_ids = client.organizations.map { \|org\| org.id.to_s }
12		- if org_ids.include?(github_org_id.to_s)
	11	+ org_ids = client.organizations.map { \|org\| org.id }
	12	+ if org_ids.include?(github_org_id)
13	13	github_user = User.create(name: env["omniauth.auth"].extra.raw_info.name, email: env["omniauth.auth"].extra.raw_info.email)
14	14	end
15	15	end
...	...