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

[BugFix] Fix Coverage Router #7036

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from

Conversation

deeleeramone
Copy link
Contributor

  1. Why?:

    • /coverage/command_model creates a KeyError
    • Fixing the KeyError creates a new problem, the output cannot be serialized.
  2. What?:

    • Add logic to parse the nested objects and serialize the output.
    • Includes coverage in the API when OpenBB router extensions are installed with routes.
  3. Impact:

    • OPENBB_DEV_MODE environment variable does does not need to be set to include this reference data.
    • Provides the API with model mappings that are more ready-to-use and easier to navigate.
  4. Testing Done:

    • OPENBB_DEV_MODE environment variable doesn't need to be set now.
    • Using the /coverage/command_model from the API no longer breaks.

@deeleeramone deeleeramone added platform OpenBB Platform v4 PRs for v4 labels Feb 18, 2025
@github-actions github-actions bot added bug Fix bug labels Feb 18, 2025
@deeleeramone deeleeramone requested a review from piiq February 19, 2025 00:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fix bug platform OpenBB Platform v4 PRs for v4
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant