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

Issue 821655 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Jul 31
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug

Blocking:
issue 806015



Sign in to add a comment

"Redeem Chrome Goodies" notification appears when no offers are available

Reported by n2102410...@gmail.com, Mar 14 2018

Issue description

UserAgent: Mozilla/5.0 (X11; CrOS x86_64 10176.76.0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/64.0.3282.190 Safari/537.36
Platform: 10176.76.0 (Official Build) stable-channel samus

Steps to reproduce the problem:
I just got the "Chrome Goodies: Redeem them before it is too late" notification. However, clicking on the notification does nothing. If I manually go to the Chromebook website, it shows that there are no offers available for my chromebook.

What is the expected behavior?

What went wrong?
1. Clicking on the notification does nothing (similar to https://bugs.chromium.org/p/chromium/issues/detail?id=396072)
2. You shouldn't send these notifications in the first place when there are no offers available.

Note that  issue 396072  is similar, but I figured I should file a new issue, since the notification shouldn't appear in the first place when there are no offers available. Apart from that, the old bug is evidently still not fixed, as clicking on the notification does nothing.

Did this work before? N/A 

Chrome version: 64.0.3282.190  Channel: stable
OS Version: 10176.76.0
Flash Version:
 
Components: -UI UI>Notifications
Owner: yoshiki@chromium.org
Status: Assigned (was: Unconfirmed)
yoshiki@ please triage
Cc: yoshiki@chromium.org jorgelo@chromium.org
Owner: stephenlin@chromium.org
Reassigning to the Goodies owner.
I believe we decided to rip out the notification and it's just a matter of getting it done. It's difficult/impossible to get the notification right without transmitting the ECHO code which we can't do without explicit user permission.

I'll see if I can target M68 for the change.
stephenlin@ we are seeing more reports coming in from pixelbook users is this. is there a quicker interim solution to turn these off?
Cc: stephenlin@chromium.org leecy@chromium.org
Owner: abutzier@chromium.org
Alex, maybe we can just make a quick simple change on the ECHO client to not send out any notifications, but leave most of the code in place for when you have more time to make sure it's safe to just rip out.

For instance, I think if we just make echo.checkAndSendNotifications return immediately then no notifications will ever be sent.

https://chrome-internal.googlesource.com/chromeos/chromeos-assets/+/master/echo/main.js#1049

Comment 6 by wfh@chromium.org, Mar 26 2018

Cc: wfh@chromium.org
I also got this notification today.
Status: Started (was: Assigned)
Blocking: 806015
Is this the same root cause as  issue 806015 ?
I'm making a change on the server side that should disable this message. Will on client side change next.
Note that the server change will disable the message for newly activated devices only due to some caching on the client. The client change is necessary to drop the notification completely.
Labels: Echo
Project Member

Comment 12 by bugdroid1@chromium.org, Apr 7 2018

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chromeos/chromeos-assets/+/21155950b72efac96401d61bdf1677aba82bd89f

commit 21155950b72efac96401d61bdf1677aba82bd89f
Author: Alex Butzier <abutzier@chromium.org>
Date: Sat Apr 07 02:50:50 2018

https://crrev.com/i/602659 will prevent any notifications being shown from ECHO. This fixes the immediate problem although additional clean up work is still needed. Fix will be in M67. 
Status: Fixed (was: Started)

Sign in to add a comment