Skip to content
Snippets Groups Projects

Fix legacy usd option appearing

Merged Ben requested to merge fix/legacy-usd-boosts-1341 into master
1 unresolved thread

Summary

Closes #2593 (closed)

Legacy boost prompt that allows payment in USD still appears sometimes.

Rather than refactoring down the class right now I've opted to stop it showing. We should schedule a refactor / remake of this in the future.

Steps to test

  1. Go to someone else's channel and click to Wire them on a post.
  2. Toggle over to send them USD.
  3. Exit wire prompt without making payment.
  4. Go back to your channel and click boost on your post.
  5. There should be nothing like the below image in the modal window.

image

Estimated Regression Scope

Maybe it could show at an inappropriate time, or in some circumstance the cookie works. Maybe this creator is re-used somewhere where USD is valid that I am unaware of/

Edited by Ben

Merge request reports

Checking pipeline status.

Merged by Xander MillerXander Miller 5 years ago (Feb 27, 2020 8:37pm UTC)

Loading

Pipeline #121697358 waiting for manual action

Pipeline waiting for manual action for f335566b on master

5 environments impacted.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading