Skip to content
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

Implementation: [Android] Show a special notice to users in Indonesia #4933

Conversation

anikiki
Copy link
Contributor

@anikiki anikiki commented Aug 23, 2024

Task/Issue URL: https://app.asana.com/0/1200581511062568/1208125695673129/f

Description

Added a new section in the New Tab Page and a view in the Legacy Tab Page.

Steps to test this PR

See https://app.asana.com/0/0/1208180779804579/f

UI changes

See https://app.asana.com/0/1200581511062568/1208180779804556/f

Copy link
Contributor Author

anikiki commented Aug 23, 2024

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @anikiki and the rest of your teammates on Graphite Graphite

@anikiki anikiki changed the title Added a new section in the New Tab Page and a view in the Legacy Tab Page. Implementation: [Android] Show a special notice to users in Indonesia Aug 29, 2024
@anikiki anikiki marked this pull request as ready for review August 29, 2024 18:56
@anikiki anikiki force-pushed the feature/ana/implementation_android_show_a_special_notice_to_users_in_indonesia branch from 5688150 to 2d95d94 Compare August 30, 2024 10:31
@malmstein malmstein self-assigned this Aug 30, 2024
Copy link
Contributor

@malmstein malmstein left a comment

Choose a reason for hiding this comment

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

Works as intented! Just a couple of non blockers, nice work!

@ContributesActivePlugin(
AppScope::class,
boundType = NewTabPageSectionPlugin::class,
priority = -1,
Copy link
Contributor

Choose a reason for hiding this comment

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

this should be moved to NewTabPageSectionPlugin.PRIORITY_INDONESIA_MESSAGE with a value of 5.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated 👍

@anikiki anikiki force-pushed the feature/ana/implementation_android_show_a_special_notice_to_users_in_indonesia branch from 2d95d94 to f46b492 Compare September 3, 2024 22:12
@anikiki anikiki merged commit 103aa9e into develop Sep 4, 2024
8 checks passed
@anikiki anikiki deleted the feature/ana/implementation_android_show_a_special_notice_to_users_in_indonesia branch September 4, 2024 10:17
0nko pushed a commit that referenced this pull request Sep 9, 2024
0nko pushed a commit that referenced this pull request Sep 9, 2024
0nko pushed a commit that referenced this pull request Sep 12, 2024
0nko pushed a commit that referenced this pull request Sep 17, 2024
0nko pushed a commit that referenced this pull request Sep 19, 2024
0nko pushed a commit that referenced this pull request Sep 19, 2024
0nko pushed a commit that referenced this pull request Sep 23, 2024
0nko pushed a commit that referenced this pull request Sep 23, 2024
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