From abc9dce84d0e2c9c6f18509de0944229f4cf02a8 Mon Sep 17 00:00:00 2001 From: Eric Harrison Date: Mon, 29 Sep 2014 20:28:28 -0700 Subject: [PATCH 1/2] Accept true and not just 'true' for company(:is_admin => true) --- lib/linked_in/api/query_helpers.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/linked_in/api/query_helpers.rb b/lib/linked_in/api/query_helpers.rb index a1be84fe..1b8eabbd 100644 --- a/lib/linked_in/api/query_helpers.rb +++ b/lib/linked_in/api/query_helpers.rb @@ -62,7 +62,7 @@ def company_path(options) elsif name = options.delete(:name) path += "/universal-name=#{CGI.escape(name)}" elsif is_admin = options.delete(:is_admin) - path += "?is-company-admin=#{CGI.escape(is_admin)}" + path += "?is-company-admin=#{CGI.escape(is_admin.to_s)}" else path += "/~" end From 25876cd0a169870b6a6158322f83c364bb3ab1fc Mon Sep 17 00:00:00 2001 From: Eric Harrison Date: Mon, 29 Sep 2014 20:29:57 -0700 Subject: [PATCH 2/2] add documentation for the is_admin feature that was obscured --- lib/linked_in/api/companies.rb | 1 + 1 file changed, 1 insertion(+) diff --git a/lib/linked_in/api/companies.rb b/lib/linked_in/api/companies.rb index 1110c100..9ab61565 100644 --- a/lib/linked_in/api/companies.rb +++ b/lib/linked_in/api/companies.rb @@ -25,6 +25,7 @@ module Companies # @option options [String] :type # @option options [String] :count # @option options [String] :start + # @option options [Boolean] :is_admin # @return [LinkedIn::Mash] def company(options = {}) path = company_path(options)