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

apply payment status #632

Merged
merged 1 commit into from
Sep 25, 2023
Merged

apply payment status #632

merged 1 commit into from
Sep 25, 2023

Conversation

ubbabeck
Copy link
Contributor

@ubbabeck ubbabeck commented Sep 22, 2023

Updating to reflect sdk integration breez/breez-sdk-greenlight#463.

@erdemyerebasmaz Do you think we need to change the logic to handle the cases where PayementStatus is failed?

@ubbabeck ubbabeck self-assigned this Sep 22, 2023
@ubbabeck ubbabeck force-pushed the ubbabeck-paymentstatus branch from 27e88dc to 589be6d Compare September 23, 2023 18:29
@ubbabeck ubbabeck changed the base branch from main to breez-sdk-436-463 September 23, 2023 18:35
@ubbabeck ubbabeck force-pushed the ubbabeck-paymentstatus branch from 589be6d to 12c97ca Compare September 25, 2023 08:10
Copy link
Collaborator

@erdemyerebasmaz erdemyerebasmaz left a comment

Choose a reason for hiding this comment

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

LGTM!

@ubbabeck ubbabeck merged commit f3ba7ce into breez-sdk-436-463 Sep 25, 2023
@erdemyerebasmaz erdemyerebasmaz deleted the ubbabeck-paymentstatus branch November 16, 2023 20:12
@erdemyerebasmaz erdemyerebasmaz restored the ubbabeck-paymentstatus branch November 16, 2023 20:12
@erdemyerebasmaz erdemyerebasmaz deleted the ubbabeck-paymentstatus branch November 16, 2023 20:13
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.

2 participants