Skip to content

[MBL-3134] Remove Reward Shipment Tracking FF#2798

Merged
scottkicks merged 2 commits into
mainfrom
scott/mbl-3134
Mar 17, 2026
Merged

[MBL-3134] Remove Reward Shipment Tracking FF#2798
scottkicks merged 2 commits into
mainfrom
scott/mbl-3134

Conversation

@scottkicks

Copy link
Copy Markdown
Contributor

📲 What

Cleans up this flag that has been actively rolled out at 100% since Apr. 2025

🤔 Why

🧹

👀 See

Oh wow it's gone 💨

After 🦋
simulator_screenshot_926B2C99-4EB1-4542-A6C6-F2397F67297C

✅ Acceptance criteria

  • Reward Tracking Number alert shows in Activity Feed when applicable and continues not to when it isn't

⏰ TODO

  • Remove from RemoteConfig Dashboard once this clean up is released

@scottkicks scottkicks changed the title cleanup ff [MBL-3134] Remove Reward Shipment Tracking FF Mar 17, 2026
@scottkicks scottkicks marked this pull request as ready for review March 17, 2026 16:08
@scottkicks scottkicks requested review from a team and ifosli and removed request for a team March 17, 2026 16:08

@ifosli ifosli left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, but make sure you don't delete the param from remote config, unless you want to revert to "false" behavior for any users on an old version of the app. There might still be some cleanup you can do in remote config, though!

@scottkicks

Copy link
Copy Markdown
Contributor Author

@ifosli Yea good call! The plan is to leave the RemoteConfig dashboard atleast untouched until all of this removal code is released, but also once the vast majority of users are on app versions where the flag is enabled

@scottkicks scottkicks merged commit a553c17 into main Mar 17, 2026
7 checks passed
@scottkicks scottkicks deleted the scott/mbl-3134 branch March 17, 2026 18:36
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.

2 participants