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

[Misc] Fix the variable name used for V4::Capsules #94

Merged
merged 1 commit into from
Oct 16, 2021

Conversation

invacuo
Copy link
Collaborator

@invacuo invacuo commented Oct 16, 2021

Why?

  • As of V4, the capsules endpoint to retrieve information about a specific capsule uses the id and not serial as it used to in v4
  • The code was still calling the variable capsule_serial which is not accurate
  • While this does not affect the functionality of the API, it is better to use the right variable name to avoid confusion in the future

Description

  • Changed the argument name
  • Updated v4.md readme

#### Why?
- As of V4, the capsules endpoint to retrieve information about a specific capsule uses the id and not serial as it used to in v4
- The code was still calling the variable `capsule_serial` which is not accurate
- While this does not affect the functionality of the API, it is better to use the right variable name to avoid confusion in the future

#### Description
- Changed the argument name
- Updated v4.md readme
@invacuo invacuo added the hacktoberfest HacktoberFest label Oct 16, 2021
@rodolfobandeira rodolfobandeira merged commit ba8d411 into rodolfobandeira:main Oct 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants