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

fixed bug #57

Closed
wants to merge 1 commit into from
Closed

fixed bug #57

wants to merge 1 commit into from

Conversation

vggonzal
Copy link
Contributor

@vggonzal vggonzal commented Dec 7, 2023

Github Issue: #56

Description

When we requested a cvs output format, the csv result was not properly constructed.

Overview of work done

Changed the workflow, there was an "if/else" loop that was not doing things properly.

Overview of verification done

Changed cvs test to include geometry.

Overview of integration done

NA

PR checklist:

  • Linted
  • Updated unit tests
  • Updated changelog
  • Integration testing

See Pull Request Review Checklist for pointers on reviewing this pull request

@frankinspace frankinspace linked an issue Dec 7, 2023 that may be closed by this pull request
@frankinspace frankinspace deleted the fix/issue-55 branch December 7, 2023 00:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CSV output incorrect when requesting geometry
2 participants