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

Handle uninstalling streaming payments with active/unclaimed streams #4099

Open
arrenv opened this issue Jan 14, 2025 · 0 comments · May be fixed by #4168
Open

Handle uninstalling streaming payments with active/unclaimed streams #4099

arrenv opened this issue Jan 14, 2025 · 0 comments · May be fixed by #4168
Assignees
Labels
bug Something isn't working Pagepro Issues handled by the Pagepro team

Comments

@arrenv
Copy link
Member

arrenv commented Jan 14, 2025

Teamwork task: Handle uninstalling streaming payments with active/unclaimed streams

Steps to reproduce

  1. Ensure the Streaming payments extension is enabled.
  2. Create some Streams, ensure some remain active and some have unclaimed funds.
  3. Try to fully uninstall the Streaming payments extension.
  4. Note the uninstallation modal, and the transaction error in the console.

Expected behaviour

image

Actual behaviour

  • The uninstallation modal is incorrect when uninstalling Streaming Payments.
  • The Streaming payments unisntall modal does not correctly handle when streams are active.
  • You are no being prevented from uninstalling when there are active streams.

image

image

@arrenv arrenv added bug Something isn't working Pagepro Issues handled by the Pagepro team labels Jan 14, 2025
@adam-strzelec adam-strzelec linked a pull request Jan 23, 2025 that will close this issue
@AdrianDudko AdrianDudko linked a pull request Jan 24, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Pagepro Issues handled by the Pagepro team
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants