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

Etf fix #2647

Merged

jmaslek merged 4 commits into main from etf_fix

Sep 27, 2022

Contributor

colin99d commented Sep 27, 2022

Description

Relates to #2612

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
Ensure the api still works
Check any related reports

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.

colin99d added 2 commits

September 27, 2022 11:47


          Fixes economy controller

ddbdb84


          Fixed basic etf issues

fe94057

colin99d requested a review from Chavithra

September 27, 2022 10:12


          Undid bad fix

9dd3d77

colin99d added the bug label

jmaslek approved these changes

View reviewed changes


          Merge branch 'main' into etf_fix

aff4a3b

jmaslek merged commit 60c060a into main

colin99d deleted the etf_fix branch

October 4, 2022 11:27

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

Labels

bug