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

Implement IssuedOrNative struct #109

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from
Open

Conversation

kevin-pease
Copy link
Contributor

✨ What kind of change does this PR introduce? (Bug fix, feature, docs update...)

Fixes #108.

  • Moves the IssuedOrNative struct to request models, making it available throughout the crate;
  • Updates several fields in request structs in 2 endpoints, so that they use this new IssuedOrNative struct;
  • Updates relevant parameter building methods;
  • Adds a few tests to test new parameter buidling;
  • Updates relevant documentation code examples.

Note: One commit in this PR is dedicated to a large set of formatting corrections. This is because of a pre-commit hook running cargo fmt.

🤔 Checklist before submitting

  • All projects build
  • Follows style guide lines
  • Relevant documentation was updated
  • Rebased onto current main

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.

Fix incorrect field types in several request structs
1 participant