Skip to content

Conversation

@jlplks
Copy link
Contributor

@jlplks jlplks commented Jul 4, 2025

📲 What

Hide collection plan when FF off and choose another reward is selected for a pledge in full payment selected before

🤔 Why

The choose another reward flow should not allow changing collection plans.

🛠 How

Fixed the validation for show collection plan or not adding the FF for edit pledge to the equation

SEE

video_proof.webm

📋 QA

Story 📖

MBL-2594

@codecov-commenter
Copy link

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

Attention: Patch coverage is 20.00000% with 4 lines in your changes missing coverage. Please review.

Project coverage is 67.26%. Comparing base (916f916) to head (354b569).

Files with missing lines Patch % Lines
...i/activities/compose/projectpage/CheckoutScreen.kt 0.00% 4 Missing ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff            @@
##             master    #2368   +/-   ##
=========================================
  Coverage     67.26%   67.26%           
- Complexity     2340     2341    +1     
=========================================
  Files           375      375           
  Lines         26658    26658           
  Branches       4030     4031    +1     
=========================================
+ Hits          17931    17932    +1     
  Misses         6544     6544           
+ Partials       2183     2182    -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jlplks jlplks merged commit bdd1a70 into master Jul 4, 2025
3 checks passed
@jlplks jlplks deleted the jpulido/MBL-2594 branch July 4, 2025 01:27
leighdouglas pushed a commit that referenced this pull request Jul 4, 2025
…uld not allow changing collection plans. (#2368)

fix collection plan for FF off
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.

4 participants