Skip to content

Conversation

@leighdouglas
Copy link
Contributor

📲 What

We discovered an edge case - if a creator sends out a survey without turning on address collection unintentionally, and the backer submits the survey without their address, but their backing includes a shippable reward, the address card will be blank. We shouldnt hide the modal, but alert the user to notify the creator to resolve the issue.

🤔 Why

This is a very very small edge case that really shouldn't happen, and hasn't really happened, but could happen, so we should account for it.

🛠 How

When the address on the backing is null, we surface text to the backer notifying them to contact the creator.

👀 See

Screenshot 2025-07-23 at 18 34 48

📋 QA

This is a complex edge to QA, but PM and I can try and find a test account with a pledge in this state.

Story 📖

MBL-2609: Edge case - empty address modal on survey_submitted cards when creator has not turned on address collection

@codecov-commenter
Copy link

codecov-commenter commented Jul 24, 2025

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

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (master@b0f6dc3). Learn more about missing BASE report.
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff            @@
##             master    #2387   +/-   ##
=========================================
  Coverage          ?   66.38%           
  Complexity        ?     2349           
=========================================
  Files             ?      378           
  Lines             ?    27904           
  Branches          ?     4230           
=========================================
  Hits              ?    18524           
  Misses            ?     7118           
  Partials          ?     2262           

☔ 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.

@leighdouglas leighdouglas marked this pull request as ready for review July 28, 2025 11:23
@leighdouglas leighdouglas merged commit a93894c into master Jul 29, 2025
3 checks passed
@leighdouglas leighdouglas deleted the leigh/mbl-2609-ec-address branch July 29, 2025 13:15
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.

5 participants