MOB-27408: add App Cards#8
Open
damlayagmur wants to merge 2 commits into
Open
Conversation
damlayagmur
requested review from
AktasBedirInsider,
heysem-useinsider,
melih-useinsider and
ozgur-useinsider
and removed request for
heysem-useinsider
ozgur-useinsider
approved these changes
May 5, 2026
melih-useinsider
approved these changes
May 5, 2026
AktasBedirInsider
approved these changes
May 6, 2026
- Add `if (!mounted) return;` guards after the await in `loadAppCards()` to prevent `setState() called after dispose()` exceptions when the user navigates away while the campaigns request is in flight. - Remove unused `url_launcher` import (package was not in pubspec and the symbol was never referenced) and unnecessary private path import `flutter_insider/src/app_cards_models.dart` (public API is sufficient). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
![insider-security-ai[bot]](https://avatars.githubusercontent.com/u/19927042?s=60&v=4){kind=small}
There was a problem hiding this comment.
✅ AI Security Review — No Issues Found
This PR adds an App Cards UI page (Flutter client-side code), updates build tool versions, and modifies the Android manifest with formatting changes. No security-relevant vulnerabilities were introduced — the changes are client-side UI, dependency version bumps, and formatting adjustments with no secrets, auth changes, or injection risks.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
flutter_insider 5.0.1.MessageCenterInboxPage.dart) that lists campaigns, supports view/click tracking, mark as read/unread, single & bulk delete, image carousel, dynamic action buttons, and deep-link handling.8.9.1and Gradle wrapper to8.12; addsINTERNETpermission andPROCESS_TEXTqueries block toAndroidManifest.xml.Jira: MOB-27408
Test plan
🤖 Generated with Claude Code