-
Notifications
You must be signed in to change notification settings - Fork 1.2k
[Android] "What’s New" promo message: Add new message type to show a list view #7072
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Android] "What’s New" promo message: Add new message type to show a list view #7072
Conversation
… RMF called What's New. This message type uses a cards_list content structure, allowing for a title, description, and a list of items, each with its own title, description, image, and action.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
…ge_add_new_message_type_to_show_a_list_view
cmonfortep
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Few comments that I think is best if we review them together.
code looks great, it's just to discuss merging strategy around RMF.
android-design-system/design-system/src/main/java/com/duckduckgo/common/ui/view/MessageCta.kt
Outdated
Show resolved
Hide resolved
android-design-system/design-system/src/main/java/com/duckduckgo/common/ui/view/MessageCta.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/com/duckduckgo/app/browser/remotemessage/RemoteMessageMapper.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/com/duckduckgo/app/browser/remotemessage/RemoteMessageMapper.kt
Show resolved
Hide resolved
...aging-impl/src/main/java/com/duckduckgo/remote/messaging/impl/mappers/RemoteMessageMapper.kt
Outdated
Show resolved
Hide resolved
...essaging-impl/src/main/java/com/duckduckgo/remote/messaging/impl/newtab/RemoteMessageView.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/com/duckduckgo/app/browser/newtab/NewTabLegacyPageView.kt
Outdated
Show resolved
Hide resolved
...ssaging-impl/src/main/java/com/duckduckgo/remote/messaging/impl/mappers/JsonActionMappers.kt
Show resolved
Hide resolved
...g-impl/src/main/java/com/duckduckgo/remote/messaging/impl/mappers/JsonRemoteMessageMapper.kt
Show resolved
Hide resolved
cmonfortep
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
All test scenarios worked well (tested with production config, attached configs, and mixing them)

Task/Issue URL: https://app.asana.com/1/137249556945/project/488551667048375/task/1211783737656574?focus=true
Description
Added support for a new "What's New" message type in the remote messaging system. This includes:
REMOTE_WHATS_NEW_MESSAGEtype to theMessageCtaclass.CardsListcontent type for displaying feature cards.UrlInContextaction type to open URLs in a contextual webview.Steps to test this PR
See Asana for more details.
NO UI changes