New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 767212 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Sep 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

Remove (beta) string to playstore

Project Member Reported by lgcheng@google.com, Sep 20 2017

Issue description

Based on request in
https://b.corp.google.com/issues/65853777

Remove beta string of 

IDS_SETTINGS_ANDROID_APPS_TITLE : settings page label
IDS_ARC_PLAYSTORE_ICON_TITLE_BETA : app launcher icon
IDS_ARC_MANAGED_PROVISION_NOTIFICATION_TITLE : notification for silent auth in managed cases
 

Comment 1 by lgcheng@google.com, Sep 20 2017

Labels: M-61 M-62

Comment 2 by lgcheng@google.com, Sep 20 2017

Labels: Merge-Request-61 Merge-Request-62

Comment 3 by lgcheng@google.com, Sep 20 2017

Cc: bhthompson@chromium.org
Bernie@ would you help approve the merge request for string change?
Project Member

Comment 4 by sheriffbot@chromium.org, Sep 20 2017

Labels: -Merge-Request-62 Merge-Review-62 Hotlist-Merge-Review
This bug requires manual review: M62 has already been promoted to the beta branch, so this requires manual review
Please contact the milestone owner if you have questions.
Owners: amineer@(Android), cmasso@(iOS), bhthompson@(ChromeOS), abdulsyed@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
We can merge this but we need to verify we are ok without doing any translations.
Project Member

Comment 6 by bugdroid1@chromium.org, Sep 21 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/680c0735a12108841ee0b5a762e9ae41b88b2b4d

commit 680c0735a12108841ee0b5a762e9ae41b88b2b4d
Author: lgcheng <lgcheng@google.com>
Date: Wed Sep 20 23:59:46 2017

Remove beta string.

Remove beta string of Play Store and Google Play Store.

TBR=dpapad@chromium.org, cpu@chromium.org

Bug:  767212 
Test: manul.
Change-Id: I99fbab7d985c4658da12b833867cdfd524438074
Reviewed-on: https://chromium-review.googlesource.com/676101
Commit-Queue: Long Cheng <lgcheng@google.com>
Reviewed-by: Long Cheng <lgcheng@google.com>
Cr-Commit-Position: refs/heads/master@{#503286}
[modify] https://crrev.com/680c0735a12108841ee0b5a762e9ae41b88b2b4d/chrome/app/chromeos_strings.grdp
[modify] https://crrev.com/680c0735a12108841ee0b5a762e9ae41b88b2b4d/chrome/app/settings_strings.grdp

Labels: -Merge-Request-61 -Merge-Review-62 Merge-Approved-62 Merge-Approved-61
Approved for 61 and 62. 
Cc: gov...@chromium.org
+govind for FYI, for 61 in this case we can get by with just the English version.
Project Member

Comment 9 by bugdroid1@chromium.org, Sep 21 2017

Labels: -merge-approved-62 merge-merged-3202
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/3e20ad53543c1b6d2b40b66527e77827d1c69b44

commit 3e20ad53543c1b6d2b40b66527e77827d1c69b44
Author: lgcheng <lgcheng@google.com>
Date: Thu Sep 21 17:36:00 2017

Remove beta string.

Remove beta string of Play Store and Google Play Store.

TBR=dpapad@chromium.org, cpu@chromium.org

Bug:  767212 
Test: manul.
Change-Id: I99fbab7d985c4658da12b833867cdfd524438074
Reviewed-on: https://chromium-review.googlesource.com/676101
Commit-Queue: Long Cheng <lgcheng@google.com>
Reviewed-by: Long Cheng <lgcheng@google.com>
Cr-Original-Commit-Position: refs/heads/master@{#503286}(cherry picked from commit 680c0735a12108841ee0b5a762e9ae41b88b2b4d)
Reviewed-on: https://chromium-review.googlesource.com/676952
Cr-Commit-Position: refs/branch-heads/3202@{#378}
Cr-Branched-From: fa6a5d87adff761bc16afc5498c3f5944c1daa68-refs/heads/master@{#499098}
[modify] https://crrev.com/3e20ad53543c1b6d2b40b66527e77827d1c69b44/chrome/app/chromeos_strings.grdp
[modify] https://crrev.com/3e20ad53543c1b6d2b40b66527e77827d1c69b44/chrome/app/settings_strings.grdp

Project Member

Comment 10 by bugdroid1@chromium.org, Sep 21 2017

Labels: -merge-approved-61 merge-merged-3163
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d0288bcea9a9bc74a6d149b445f39ce547f804c0

commit d0288bcea9a9bc74a6d149b445f39ce547f804c0
Author: lgcheng <lgcheng@google.com>
Date: Thu Sep 21 17:38:18 2017

Remove beta string.

Remove beta string of Play Store and Google Play Store.

TBR=dpapad@chromium.org, cpu@chromium.org

Bug:  767212 
Test: manul.
Change-Id: I99fbab7d985c4658da12b833867cdfd524438074
Reviewed-on: https://chromium-review.googlesource.com/676101
Commit-Queue: Long Cheng <lgcheng@google.com>
Reviewed-by: Long Cheng <lgcheng@google.com>
Cr-Original-Commit-Position: refs/heads/master@{#503286}(cherry picked from commit 680c0735a12108841ee0b5a762e9ae41b88b2b4d)
Reviewed-on: https://chromium-review.googlesource.com/677072
Cr-Commit-Position: refs/branch-heads/3163@{#1254}
Cr-Branched-From: ff259bab28b35d242e10186cd63af7ed404fae0d-refs/heads/master@{#488528}
[modify] https://crrev.com/d0288bcea9a9bc74a6d149b445f39ce547f804c0/chrome/app/chromeos_strings.grdp
[modify] https://crrev.com/d0288bcea9a9bc74a6d149b445f39ce547f804c0/chrome/app/settings_strings.grdp

Comment 11 by lgcheng@google.com, Sep 21 2017

Status: Fixed (was: Started)

Sign in to add a comment