Skip to content

chore: clean up FavoritesByStop feature flag - #1914

Open
KaylaBrady wants to merge 1 commit into
mainfrom
kb-remove-group-by-favoriets
Open

chore: clean up FavoritesByStop feature flag#1914
KaylaBrady wants to merge 1 commit into
mainfrom
kb-remove-group-by-favoriets

Conversation

@KaylaBrady

Copy link
Copy Markdown
Collaborator

Summary

Ticket: Release favorites grouped by stop

What is this PR for?
Remove the feature flag & unused code paths that relied on the flag being false

iOS

  • If you added any user-facing strings on iOS, are they included in Localizable.xcstrings?
    • Add temporary machine translations, marked "Needs Review"

android

  • All user-facing strings added to strings resource in alphabetical order
  • Expensive calculations are run in withContext(Dispatchers.Default) where possible (ideally in shared code)

Testing

What testing have you done?

@KaylaBrady
KaylaBrady marked this pull request as ready for review August 5, 2026 13:44
@KaylaBrady
KaylaBrady requested a review from a team as a code owner August 5, 2026 13:44
@KaylaBrady
KaylaBrady requested a review from thedecaftony August 5, 2026 13:44
@KaylaBrady
KaylaBrady force-pushed the kb-remove-group-by-favoriets branch from d8b0575 to ab9db01 Compare August 5, 2026 13:46
@KaylaBrady
KaylaBrady added this pull request to the merge queue Aug 6, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to no response for status checks Aug 6, 2026
@KaylaBrady
KaylaBrady added this pull request to the merge queue Aug 6, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to no response for status checks Aug 6, 2026
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