Skip to content

Feat : Add push notification on onchain transaction receive#290

Open
heyolaniran wants to merge 2 commits intolnflash:mainfrom
heyolaniran:send_push_notification
Open

Feat : Add push notification on onchain transaction receive#290
heyolaniran wants to merge 2 commits intolnflash:mainfrom
heyolaniran:send_push_notification

Conversation

@heyolaniran
Copy link

@heyolaniran heyolaniran commented Feb 27, 2026

🧪 Testing Coverage

  • Unit Tests: Describe tested components or logic
  • Integration Tests: State management or system interaction
  • Manual Testing: End-to-end user journey or QA notes

Hello @islandbitcoin @brh28

This PR improves the handling of on-chain payment notifications received via Ibex webhooks and closes #120

thanks for having time to review the work and let me know if I should fix something about it.

@brh28
Copy link
Contributor

brh28 commented Mar 2, 2026

@heyolaniran You have this marked as end-to-end tested, but I am curious how much of it you tested. I would be surprised/impressed if you were able to receive a push notification via firebase, especially if triggered by a real on-chain payment.

@islandbitcoin Let me know how you want to proceed with this.

@heyolaniran
Copy link
Author

hello @brh28 sorry for this, certainly a mistake by my side while I was deleting the default pull request format. I was just able to run the unit and integration test. It is unchecked for the moment.

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.

Send Push Notification upon onchain confirmation

2 participants