Merged
Conversation
Contributor
📊 Diff Coverage Report✓ Coverage check passed (no source files changed) |
added 2 commits
February 10, 2026 16:18
c694e55 to
5c4f6f6
Compare
- dont need remove all aliases button
- Resolved conflicts keeping Jetpack Compose UI - Added batch operations to Repository: addAliases, addTags, addTriggers - Added batch operations to ViewModel: removeSelectedAliases/Tags/Triggers - Fixed trackEvent signature to use Map<String, Any?>? - Removed unused XML layouts (using Compose dialogs instead) Co-authored-by: Cursor <cursoragent@cursor.com>
- Add MultiPairInputDialog for adding multiple aliases/tags/triggers - Add MultiSelectRemoveDialog for removing selected items - Update AliasesSection, TagsSection, TriggersSection with batch buttons - Wire up batch dialogs in MainScreen Co-authored-by: Cursor <cursoragent@cursor.com>
- Use known working image URLs (from V1 sample app) - Add large_icon support to NotificationType - Update Accept header to match V1 format - Add better logging for debugging API calls Co-authored-by: Cursor <cursoragent@cursor.com>
- Add Material icons to InAppMessageType (VerticalAlignTop, VerticalAlignBottom, CropSquare, Fullscreen) - Update SendInAppMessageSection to show type-specific icons - Improve sendInAppMessage toast to show "Sent In-App Message: Top Banner" etc - Update triggers list when sending IAM Co-authored-by: Cursor <cursoragent@cursor.com>
- Update project foundation to reflect Compose UI - Add batch operations documentation (addAliases, addTags, etc.) - Add REST API key requirement for notifications - Update section descriptions with new batch buttons - Add Track Event JSON validation details - Add Send IAM icons documentation - Update file structure for Compose architecture - Add Phase 8: Jetpack Compose Architecture section - Document reusable components, theme, and toast messages Co-authored-by: Cursor <cursoragent@cursor.com>
fadi-george
approved these changes
Feb 12, 2026
Contributor
|
Should squash and maybe also delete old example |
4b3ef21 to
38855d6
Compare
Contributor
|
we should keep the old one unless the new one is already hooked up into the SDK |
df6b37a to
b5c643a
Compare
72f1d79 to
2025b53
Compare
2025b53 to
771e7dc
Compare
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
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.
Description
One Line Summary
Affected code checklist
Checklist
Overview
Testing
Final pass
This change is