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

docs: fix the documentation of the balance #202

Merged
merged 4 commits into from
Apr 20, 2020
Merged

docs: fix the documentation of the balance #202

merged 4 commits into from
Apr 20, 2020

Conversation

vrolland
Copy link
Contributor

No description provided.

@coveralls
Copy link

coveralls commented Apr 20, 2020 •
edited
Loading

Coverage Status

Coverage remained the same at 90.353% when pulling f817ab2 on balance-error-managing into 17cf07e on master.

benjlevesque
benjlevesque approved these changes Apr 20, 2020
View reviewed changes
Copy link
Contributor

@benjlevesque benjlevesque left a comment

Choose a reason for hiding this comment

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

1 important comment in API docs (don't use getData), otherwise details

@vrolland @benjlevesque
Update packages/docs/docs/guides/3-Portal-API/2-payment-status.md
f817ab2 
Co-Authored-By: Benjamin LEVESQUE <14175665+benjlevesque@users.noreply.github.com>
@vrolland vrolland merged commit 7b7913b into master Apr 20, 2020
@vrolland vrolland deleted the balance-error-managing branch April 20, 2020 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@benjlevesque benjlevesque benjlevesque approved these changes

@rittme rittme Awaiting requested review from rittme

@yomarion yomarion Awaiting requested review from yomarion

@AdamREQ AdamREQ Awaiting requested review from AdamREQ

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@vrolland @coveralls @benjlevesque @AdamREQ