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

Document account deletion #217

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

waldyrious
Copy link
Contributor

  • Update Account-related information in the Entities section
  • Document functionality to delete accounts

@waldyrious waldyrious self-assigned this Jul 2, 2021
@waldyrious waldyrious requested review from nunofgs and fmfmartins July 2, 2021 18:38
@waldyrious waldyrious force-pushed the feature/delete-accounts branch from 7df83f1 to 56e97ad Compare July 3, 2021 18:04
_entities.md Outdated
@@ -20,12 +21,12 @@
Property | Description
-------- | -----------------------------------------------------------------------
billing | The relevant billing details associated with the account.
brand | The brand of the `card` account.
brand | The brand of the account (if it is of type `card`).
currency | The currency in which the account is denominated.
id | A unique ID associated with the account.
label | The display name of the account as chosen by the user.
status | The current status of the account. Possible values are `ok` and `failed`.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
status | The current status of the account. Possible values are `ok` and `failed`.
status | The current status of the account. Possible values are `blocked`, `expired`, `failed`, `ok` and `pending`.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added to 81f66f3.

@waldyrious waldyrious force-pushed the feature/delete-accounts branch from 56e97ad to ef5c39b Compare July 5, 2021 14:05
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