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.

Sign up for GitHub

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

Jump to bottom

Fixed crypto report issues #2722

Merged

colin99d merged 1 commit into main from fix_2679

Oct 6, 2022

Contributor

colin99d commented Oct 6, 2022 •

edited

Loading

Description

Fixes #2679

How has this been tested?

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 (not applicable)
Ensure the api still works (not applicable)
Check any related reports
- python terminal.py reports/crypto BTC

Checklist:

Update our Hugo documentation following these guidelines.
Update our tests following these guidelines.
Make sure you are following our CONTRIBUTING guidelines.
If a feature was added make sure to add it to the corresponding scripts file.

Others

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


          Fixed crypto report issues

7717dd4

colin99d added the bug label

jose-donato approved these changes

View reviewed changes

Contributor

jose-donato left a comment

wonderful work as always colin

colin99d merged commit 90467fd into main

colin99d deleted the fix_2679 branch

October 6, 2022 10:16

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug