Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for balance sheet API endpoint #91

Merged
merged 6 commits into from
Nov 9, 2020

Conversation

tylerhaugen-stanley
Copy link
Collaborator

No description provided.

@tylerhaugen-stanley
Copy link
Collaborator Author

Pretty sure I didn't handle amending the commits properly, let me know if you'd like to me fix things.

@@ -0,0 +1,53 @@
---
http_interactions:
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I ended up copying the majority of these fixture files from other examples but based o n the various dates throughout the files it seems as though a generator of sorts was used. Please advise if I should do things differently than I have here.

Copy link
Owner

@dblock dblock Nov 3, 2020

Choose a reason for hiding this comment

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

These are not using a generator but recorded using vcr against production iex endpoints. Delete the .yml files you copied, set the tokens in your ENV and run rspec ...spec.rb without the presence of the VCR file and it will record new interactions.

If the file (e.g. msft.yml) exists, it will be used and no real HTTP request will be made. Otherwise an HTTP request is made. Also take a look at https://github.com/dblock/iex-ruby-client/blob/master/spec/support/vcr.rb of how we prevent your tokens from being written into these.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Cool thanks for the info. I currently don't have a paid IEX account so I don't believe this will work for me since balance sheet is a premium endpoint. This seems like a common issue I imagine you have come across in the past, is there a good way around this?

Copy link
Owner

Choose a reason for hiding this comment

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

i did, no way around this, i just pay for it - maybe email them and they can give you some credits to contribute here?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I'll just pay for the account i was planning to eventually anyway. Will look into doing this properly this weekend.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Updated with proper fixture files.

@dblock
Copy link
Owner

dblock commented Nov 8, 2020

Looks good, needs a rebase.

@dblock dblock merged commit 935ba53 into dblock:master Nov 9, 2020
@tylerhaugen-stanley tylerhaugen-stanley deleted the add-balance-sheet branch November 19, 2020 02:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants