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

Error early when params not specified for card functions #484

Merged
merged 2 commits into from
Oct 24, 2017

Commits on Oct 23, 2017

  1. Error early when params not specified for card functions

    Card-related API calls are a bit of an unusual case because we need to
    have an instantiated params struct in order to determine the correct URL
    to which to make the request. Previously, if a `nil` was passed to any
    of these functions, we'd panic as we tried to access one of its members.
    
    This patch just makes the params requirement a little more explicit by
    returning an error if any of card functions is called with `nil` params.
    
    Fixes #483.
    brandur committed Oct 23, 2017
    Configuration menu
    Copy the full SHA
    6aacada View commit details
    Browse the repository at this point in the history

Commits on Oct 24, 2017

  1. Configuration menu
    Copy the full SHA
    ce3b0b6 View commit details
    Browse the repository at this point in the history