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.

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

Fix error in VAN Target Export method. #704

Merged
merged 3 commits into from
Jul 8, 2022
Merged

Fix error in VAN Target Export method. #704

merged 3 commits into from
Jul 8, 2022

Commits on Jun 24, 2022

  1. Fix error VAN Target Export method. 

    It looks like this method assumes VAN return a *list* of exported files
rather than a single target, but VAN currently only returns individual
targets: https://docs.ngpvan.com/reference/targetexportjobsexportjobid

They might've changed the API since this was last updated? In any event,
these tweaks return correct data in my recent testing!
    @cmc333333
    cmc333333 committed Jun 24, 2022
    Configuration menu
    Copy the full SHA
    785a768 View commit details
    Browse the repository at this point in the history

  2. Lint

    @cmc333333
    cmc333333 committed Jun 24, 2022
    Configuration menu
    Copy the full SHA
    0dbf1c1 View commit details
    Browse the repository at this point in the history

Commits on Jun 29, 2022

  1. Let petl stream target results.

    @cmc333333
    cmc333333 committed Jun 29, 2022
    Configuration menu
    Copy the full SHA
    eae7edf View commit details
    Browse the repository at this point in the history