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

Minor fixes in API usage #2588

Merged
merged 6 commits into from
Sep 22, 2022
Merged

Minor fixes in API usage #2588

merged 6 commits into from
Sep 22, 2022

Conversation

montezdesousa
Copy link
Contributor

@montezdesousa montezdesousa commented Sep 22, 2022

Description

This PR fixes some small details on the api and reverts default plot style to boring

openbb.economy.get_ycrv_countries()
openbb.economy.get_events_countries()

import datetime
openbb.stocks.candle("GME", add_trend=True, ma=[20, 50, 100], start_date=datetime.datetime(2020, 10, 18), asset_type="This is")

@montezdesousa montezdesousa marked this pull request as ready for review September 22, 2022 10:04
@montezdesousa montezdesousa added the do not merge Label to prevent pull request merge label Sep 22, 2022
@montezdesousa montezdesousa removed the do not merge Label to prevent pull request merge label Sep 22, 2022
@montezdesousa montezdesousa merged commit 16b2a7a into main Sep 22, 2022
@Chavithra Chavithra deleted the api_cleanup branch September 26, 2022 13:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fix bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants