Skip to content
This repository has been archived by the owner on Aug 2, 2021. It is now read-only.

[stock-search] IEX API Update #24

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

2ooLe
Copy link

@2ooLe 2ooLe commented Sep 27, 2019

Appears the IEX API now requires authentication. Updated various items to make the script functional.

Copy link

@AirWick219 AirWick219 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change works but can you also update the README.md here to includes gem install iex-ruby-client

client = IEX::Api::Client.new(
publishable_token: "#{token}",
)
quote = client.quote("#{parsed_symbol}")

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you can just make quote = client.quote(parsed_symbol)

quote = JSON.parse(response)

client = IEX::Api::Client.new(
publishable_token: "#{token}",
Copy link

@almokhtarbr almokhtarbr Sep 13, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also here just like that publishable_token: token.to_s,

@AirWick219
Copy link

@almokhtarbr Ima not sure anyone maintains this repo. I forked it and made some changes there.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants