You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The initial work to select and integrate a Stock API has been accomplished. We will be using MarketStack, which provides 1,000 free stock price updates a month under a free plan. If our stakeholder wants more updates, we can talk to her about purchasing a paid plan.
The initial implementation isn't lengthy, but/and needs some re-architecting.
This issue can be completed in stages (multiple PRs) or all together. You may want to go create a free MarketStack account for yourself. It does not require you enter any financial info.
More refactoring. Here's one relevant comment. Personally, I don't think any of this should be in models/, and may be better namespaced in an app/api/ directory where the classes are Api::Stock/etc.
@malsmr - Absolutely! - Please feel free to do some of the bullets and document any constraints concerns in regards to obtaining any others. Accomplishing this in multiple deliverables is fine by me.
Summary
The initial work to select and integrate a Stock API has been accomplished. We will be using MarketStack, which provides 1,000 free stock price updates a month under a free plan. If our stakeholder wants more updates, we can talk to her about purchasing a paid plan.
The initial implementation isn't lengthy, but/and needs some re-architecting.
This issue can be completed in stages (multiple PRs) or all together. You may want to go create a free MarketStack account for yourself. It does not require you enter any financial info.
See criteria.
Things to Consider
#32
#37
#55
Criteria for Completion
Faraday
instead ofNet::HTTP
, as mentioned heremodels/
, and may be better namespaced in anapp/api/
directory where the classes areApi::Stock
/etc.The text was updated successfully, but these errors were encountered: