OpenBB icon indicating copy to clipboard operation
OpenBB copied to clipboard

Bug fix for ETF export

Open victortangggg opened this issue 3 years ago • 1 comments

Description

  • [ ] Summary of the change / bug fix.
  • [ ] Link # issue, if applicable.
  • [ ] Screenshot of the feature or the bug before/after fix, if applicable.
  • [ ] Relevant motivation and context.
  • [ ] List any dependencies that are required for this change.

fix bug for ETF export by using correct data type for parameters Link #3654

image

How has this been tested?

manually

  • Please describe the tests that you ran to verify your changes.
  • Provide instructions so we can reproduce.
  • Please also list any relevant details for your test configuration.
  • [ ] Make sure affected commands still run in terminal
  • [ ] Ensure the SDK still works
  • [ ] Check any related reports

N/A

Checklist:

Others

  • [ ] I have performed a self-review of my own code.
  • [ ] I have commented my code, particularly in hard-to-understand areas.

victortangggg avatar Nov 30 '22 12:11 victortangggg

Looks like the linter is failing. You can fix that by running black ./.

jmaslek avatar Nov 30 '22 14:11 jmaslek

@jmaslek hey thanks for helping. I think the linting passes now! :)

victortangggg avatar Nov 30 '22 19:11 victortangggg