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

Use Greaselion for YouTube publishers #9133

Closed
NejcZdovc opened this issue Apr 10, 2020 · 1 comment · Fixed by brave/brave-core#5440
Closed

Use Greaselion for YouTube publishers #9133

NejcZdovc opened this issue Apr 10, 2020 · 1 comment · Fixed by brave/brave-core#5440

Comments

@NejcZdovc
Copy link
Contributor

NejcZdovc commented Apr 10, 2020

we should switch to GL for YT so that we can have faster turn around rate when something breaks.

@emerick emerick added the QA/Yes label Aug 3, 2020
@NejcZdovc NejcZdovc changed the title Use greaselion for publisher flows Use GreaseLion for YouTube publishers Aug 27, 2020
@NejcZdovc NejcZdovc added this to the 1.15.x - Nightly milestone Aug 27, 2020
@srirambv srirambv changed the title Use GreaseLion for YouTube publishers [Desktop] Use GreaseLion for YouTube publishers Sep 9, 2020
@LaurenWags
Copy link
Member

LaurenWags commented Sep 28, 2020

Verification passed using below builds

Brave	1.16.42 Chromium: 86.0.4240.55 (Official Build) nightly (x86_64)
Revision	a6d625ef6f7fe8ea0675f1cf759155a05ee1be40-refs/branch-heads/4240@{#953}
OS	macOS Version 10.14.6 (Build 18G3020)
General
  • Confirmed Greaselion script no longer needs to be installed manually, just need to confirm that Brave Local Data Updater is version >=1.0.27.
Component
Channel URL channel - sampson channel - ramsay channel 2 channel - 3 channel - custom banner
User URL user - verified user - unverified user 2 user3 user - custom banner
Activity

Logged #11931, #11932 as an intermittent issue observed where the AC list isn't always as expected for video activity.

Also encountered #11933 for AC list not populating as expected when opening YT videos via right click.

Ran thru steps from brave/brave-core#5440, substituted a shorter video (https://www.youtube.com/watch?v=XoxrQr-w8Kg). Video length was 1min 33s or 93s which matches the duration in the DB.

Nightly DB - Activity TC Nightly AC table - Activity TC
Brave	1.16.45 Chromium: 86.0.4240.55 (Official Build) nightly (x86_64)
Revision	a6d625ef6f7fe8ea0675f1cf759155a05ee1be40-refs/branch-heads/4240@{#953}
OS	macOS Version 10.14.6 (Build 18G3020)
Video URL
  • Verified publisher (using staging env):
    • Confirmed tip to https://www.youtube.com/watch?v=uWKCm2GKVyE showed on Tips page of brave://rewards as expected
    • Confirmed recurring tip showed on Monthly Contributions panel of brave://rewards as expected
    • Confirmed able to adjust monthly contribution amount from panel
    • Confirmed able to remove monthly contribution from panel and brave://rewards page
    • Confirmed panel, 1 time tip, and recurring tip banners all had verified checkmark
    • Watched a video for a bit and
      • Confirmed panel showed expected favicon and %
      • Confirmed showed on AC table and % value was as expected
  • Non-verified publisher (using staging env):
    • Confirmed tip to https://www.youtube.com/watch?v=9i4SKHbhbqk showed on pending table
    • Confirmed recurring tip showed on Monthly Contributions panel of brave://rewards as expected
    • Confirmed able to adjust monthly contribution amount from panel
    • Confirmed able to remove monthly contribution from panel and brave://rewards page
    • Watched a video for a bit and
      • Confirmed panel showed expected favicon and %
      • Confirmed showed on AC table and % value was as expected
  • Custom banner (using prod env)
video - verified video - unverified video - AC video - UI1 video - UI2 video custom  banner1 video custom banner2
Brave	1.16.46 Chromium: 86.0.4240.55 (Official Build) nightly (x86_64)
Revision	a6d625ef6f7fe8ea0675f1cf759155a05ee1be40-refs/branch-heads/4240@{#953}
OS	macOS Version 10.14.6 (Build 18G3020)
Custom URL
  • Verified publisher (using production env):
    • Confirmed tip to https://www.youtube.com/vice showed on Tips page of brave://rewards as expected
    • Confirmed recurring tip showed on Monthly Contributions panel of brave://rewards as expected
    • Confirmed able to adjust monthly contribution amount from panel
    • Confirmed able to remove monthly contribution from panel and brave://rewards page
    • Confirmed panel, 1 time tip, and recurring tip banners all had verified checkmark
  • Non-verified publisher (using production env):
    • Confirmed tip to https://www.youtube.com/ninja showed on pending table
    • Confirmed recurring tip showed on Monthly Contributions panel of brave://rewards as expected
    • Confirmed able to adjust monthly contribution amount from panel
    • Confirmed able to remove monthly contribution from panel and brave://rewards page
  • Custom banner (using prod env) - note, I was unable to find a verified YT publisher with a custom URL and a fully customized banner similar to examples above. However, in the example below the banner text was customized and is different from the default text.
custom - vice - verified custom - ninja - unverified custom UI 1 custom UI 2 custom - custom banner
History
  • Verified steps from Rewards GreaseLion - YouTube brave-core#5440
  • Confirmed after navigating to various youtube URLs in the same tab the panel displayed the expected publisher info. Used a mix of verified and non-verified publishers
  • Used backward/forward navigation in the same tab and confirmed the panel showed the expected publisher info.
Tab Activation
  • Verified steps from Rewards GreaseLion - YouTube brave-core#5440
  • Opened various YT URLs in multiples tabs and confirmed the panel displayed the expected publisher info. Used a mix of verified and non-verified publishers
  • Switched between the various tabs and confirmed the panel showed the expected publisher info.
Tipping non-YouTube publishers

Regular site:
Verified Site

GitHub:
verified - GH

Twitch:
verified - twitch

Vimeo (broken):
verified - vimeo (broken)

Reddit:
verified - reddit

Twitter:
verified - twitter

One time and recurring tips:
Screen Shot 2020-09-30 at 7 51 09 AM

Other Various Tests - Monthly Contributions
  • Set up monthly tips for each of the following types of verified publishers: YouTube, regular sites, GitHub, Reddit, Twitter, Twitch. Confirmed at expected time the monthly contributions were made:
monthly contribs monthly contribs 2 monthly contribs 3 monthly contribs 4
Other Various Tests - panel and banner messaging
  • Confirmed non-verified YT publishers show specific messaging on panel and tipping banners
non-verified Screen Shot 2020-09-30 at 8 18 19 AM Screen Shot 2020-09-30 at 8 18 32 AM
  • Confirmed verified, non KYC'd YT publishers show different messaging on panel and tipping banners
verified, non-KYCd Screen Shot 2020-09-30 at 8 18 44 AM Screen Shot 2020-09-30 at 8 18 55 AM
  • Confirmed verified, KYC'd YT publishers do not show any messaging on panel and tipping banners
verified, KYCd Screen Shot 2020-09-30 at 8 19 05 AM Screen Shot 2020-09-30 at 8 19 15 AM
Other Various Tests - Settings
  • Modified AC settings to disable "Allow contribution to videos". Viewed some YT videos for several minutes each. Confirmed no YT videos were added to AC table.
  • Re-enabled "Allow contribution to videos" and visited a new YT video. Confirmed the publisher was added to AC list.
  • Confirmed able to exclude a YT publisher from AC via the toggle on the panel
  • Confirmed able to exclude a YT publisher from AC via the AC list on brave://rewards
  • Confirmed able to exclude a YT publisher from AC via the popup list of AC sites from brave://rewards
  • Confirmed able to remove a YT publisher from the exclude list and have the publisher get re-added to AC
Brave	1.16.47 Chromium: 86.0.4240.55 (Official Build) dev (x86_64)
Revision	a6d625ef6f7fe8ea0675f1cf759155a05ee1be40-refs/branch-heads/4240@{#953}
OS	macOS Version 10.14.6 (Build 18G3020)
Upgrade

Using 1.15.65 (last 1.15.x Dev version), set up the following data:

  • Have a few 1 time tips to verified publishers completed (some to YT publishers, some other publisher types also)
  • Have a few pending 1 time tips to non-verified YT publishers
  • Have a few recurring tips to verified publishers completed (some to YT publishers, some other publisher types also)
  • Have AC list populated with a mix of verified and non-verified sites, a mix of publisher types including YT publishers

Upgrade to 1.16.x (on Dev channel) version with Greaselion YT tipping.

  • Confirm all data is preserved as expected on brave://rewards.
  • Confirm if you navigate to the various publishers and open the panel, the AC % is preserved (if publisher is part of AC list), the recurring tip amount is preserved (if set up). Confirm panel shows expected information (favicon if applicable, publisher name, etc). Confirm custom tip banner is available (if applicable)
  • Confirm if you view another video by a publisher already on AC list the values in the DB (I looked at duration/visits on activity_info) and AC % change appropriately
  • Confirm you can view a YT video by a publisher not already on AC list and it gets added to AC list
  • For an existing recurring tip, confirm you are able to adjust the recurring tip amount for YT publisher from the panel
  • For an existing recurring tip, confirm you are able to delete a recurring tip for YT publisher

Verification passed on

Brave 1.16.55 Chromium: 86.0.4240.72 (Official Build) dev (64-bit)
Revision 581582174c512f44f44fd1aea340471f54b2365f-refs/branch-heads/4240@{#1134}
OS Ubuntu 18.04 LTS
General
  • Confirmed Greaselion script no longer needs to be installed manually, just need to confirm that Brave Local Data Updater is version >=1.0.27.

image

Channel URL

image
image
image
image
image

User URL
  • Verified steps from Rewards GreaseLion - YouTube brave-core#5440, used staging env
  • Verified publisher:
    • Confirmed tip to https://www.youtube.com/user/JJDSampson showed on Tips panel of brave://rewards as expected
    • Confirmed recurring tip showed on Monthly Contributions panel of brave://rewards as expected
    • Confirmed able to adjust monthly contribution amount from panel
    • Confirmed able to remove monthly contribution from panel and brave://rewards page
    • Confirmed panel, 1 time tip, and recurring tip banners all had verified checkmark
  • Non-verified publisher:
    • Confirmed tip to https://www.youtube.com/user/gordonramsay showed on pending table
    • Confirmed recurring tip showed on Monthly Contributions panel of brave://rewards as expected
    • Confirmed able to adjust monthly contribution amount from panel
    • Confirmed able to remove monthly contribution from panel and brave://rewards page
  • Custom banner (used prod env)

image
image
image
image
image

Activity

Encountered #11931, #11932 as an intermittent issue observed where the AC list isn't always as expected for video activity.

Also encountered #11933 for AC list not populating as expected when opening YT videos via right click.

Ran thru steps from brave/brave-core#5440, substituted a shorter video (https://www.youtube.com/watch?v=XoxrQr-w8Kg). Video length was 1min 33s or 93s which matches the duration in the DB.

image
image

Video URL
  • Verified publisher (using staging env):
    • Confirmed tip to https://www.youtube.com/watch?v=uWKCm2GKVyE showed on Tips page of brave://rewards as expected
    • Confirmed recurring tip showed on Monthly Contributions panel of brave://rewards as expected
    • Confirmed able to adjust monthly contribution amount from panel
    • Confirmed able to remove monthly contribution from panel and brave://rewards page
    • Confirmed panel, 1 time tip, and recurring tip banners all had verified checkmark
    • Watched a video for a bit and
      • Confirmed panel showed expected favicon and %
      • Confirmed showed on AC table and % value was as expected
  • Non-verified publisher (using staging env):
    • Confirmed tip to https://www.youtube.com/watch?v=9i4SKHbhbqk showed on pending table
    • Confirmed recurring tip showed on Monthly Contributions panel of brave://rewards as expected
    • Confirmed able to adjust monthly contribution amount from panel
    • Confirmed able to remove monthly contribution from panel and brave://rewards page
    • Watched a video for a bit and
      • Confirmed panel showed expected favicon and %
      • Confirmed showed on AC table and % value was as expected
  • Custom banner (using prod env)

image
image
image
image
image

Custom URL
  • Verified publisher (using production env):
    • Confirmed tip to https://www.youtube.com/vice showed on Tips page of brave://rewards as expected
    • Confirmed recurring tip showed on Monthly Contributions panel of brave://rewards as expected
    • Confirmed able to adjust monthly contribution amount from panel
    • Confirmed able to remove monthly contribution from panel and brave://rewards page
    • Confirmed panel, 1 time tip, and recurring tip banners all had verified checkmark
  • Non-verified publisher (using production env):
    • Confirmed tip to https://www.youtube.com/ninja showed on pending table
    • Confirmed recurring tip showed on Monthly Contributions panel of brave://rewards as expected
    • Confirmed able to adjust monthly contribution amount from panel
    • Confirmed able to remove monthly contribution from panel and brave://rewards page
  • Custom banner (using prod env) - note, I was unable to find a verified YT publisher with a custom URL and a fully customized banner similar to examples above. However, in the example below the banner text was customized and is different from the default text.

image
image
image
image
image
image

History
  • Verified steps from Rewards GreaseLion - YouTube brave-core#5440
  • Confirmed after navigating to various youtube URLs in the same tab the panel displayed the expected publisher info. Used a mix of verified and non-verified publishers
  • Used backward/forward navigation in the same tab and confirmed the panel showed the expected publisher info.
Tab Activation
  • Verified steps from Rewards GreaseLion - YouTube brave-core#5440
  • Opened various YT URLs in multiples tabs and confirmed the panel displayed the expected publisher info. Used a mix of verified and non-verified publishers
  • Switched between the various tabs and confirmed the panel showed the expected publisher info.
Tipping non-YouTube publishers

Regular site:
image

GitHub:
image

Twitch:
image
Encountered #6142

Vimeo (broken):
image

Reddit:
image

Twitter:
image

One time and recurring tips:
image
image

Other Various Tests - Monthly Contributions
  • Set up monthly tips for each of the following types of verified publishers. Confirmed at expected time the monthly contributions were made:
    image
    image
Other Various Tests - panel and banner messaging
  • Confirmed non-verified YT publishers show specific messaging on panel and tipping banners
    image
    image
    image

  • Confirmed verified, non KYC'd YT publishers show different messaging on panel and tipping banners
    image
    image
    image

  • Confirmed verified, KYC'd YT publishers do not show any messaging on panel and tipping banners
    image
    image
    image

Other Various Tests - Settings
  • Modified AC settings to disable "Allow contribution to videos". Viewed some YT videos for several minutes each. Confirmed no YT videos were added to AC table.
  • Re-enabled "Allow contribution to videos" and visited a new YT video. Confirmed the publisher was added to AC list.
  • Confirmed able to exclude a YT publisher from AC via the toggle on the panel
  • Confirmed able to exclude a YT publisher from AC via the AC list on brave://rewards
  • Confirmed able to exclude a YT publisher from AC via the popup list of AC sites from brave://rewards
  • Confirmed able to remove a YT publisher from the exclude list and have the publisher get re-added to AC
Upgrade

Using 1.15.x , set up the following data:

  • Have a few 1 time tips to verified publishers completed (some to YT publishers, some other publisher types also)
  • Have a few pending 1 time tips to non-verified YT publishers
  • Have a few recurring tips to verified publishers completed (some to YT publishers, some other publisher types also)
  • Have AC list populated with a mix of verified and non-verified sites, a mix of publisher types including YT publishers

Upgrade to 1.16.x version with Greaselion YT tipping.

  • Confirm all data is preserved as expected on brave://rewards.
  • Confirm if you navigate to the various publishers and open the panel, the AC % is preserved (if publisher is part of AC list), the recurring tip amount is preserved (if set up). Confirm panel shows expected information (favicon if applicable, publisher name, etc). Confirm custom tip banner is available (if applicable)
  • Confirm if you view another video by a publisher already on AC list the values in the DB (I looked at duration/visits on activity_info) and AC % change appropriately
  • Confirm you can view a YT video by a publisher not already on AC list and it gets added to AC list
  • For an existing recurring tip, confirm you are able to adjust the recurring tip amount for YT publisher from the panel
  • For an existing recurring tip, confirm you are able to delete a recurring tip for YT publisher

Verification passed using


Brave | 1.16.51 Chromium: 86.0.4240.68 (Official Build) dev (64-bit)
-- | --
Revision | ad72ee9aa8e15ed300df1238e76c7a8f4d686f97-refs/branch-heads/4240@{#1097}
OS | Windows 10 OS Version 1903 (Build 18362.1016)

General
  • Confirmed Greaselion script no longer needs to be installed manually, just need to confirm that Brave Local Data Updater is version >=1.0.28

image


Brave | 1.16.61 Chromium: 86.0.4240.80 (Official Build) dev (64-bit)
-- | --
Revision | 7ed88b53bda45a2d19efb4f8706dd6b6cad0d3af-refs/branch-heads/4240@{#1183}
OS | Windows 10 OS Version 1903 (Build 18362.1082)

Channel URL

image
image
image
image
image
image
image

User URL
  • Verified steps from Rewards GreaseLion - YouTube brave-core#5440, used staging env
  • Verified publisher:
    • Confirmed tip to https://www.youtube.com/user/JJDSampson showed on Tips panel of brave://rewards as expected
    • Confirmed recurring tip showed on Monthly Contributions panel of brave://rewards as expected
    • Confirmed able to adjust monthly contribution amount from panel
    • Confirmed able to remove monthly contribution from panel and brave://rewards page
    • Confirmed panel, 1 time tip, and recurring tip banners all had verified checkmark
  • Non-verified publisher:
    • Confirmed tip to https://www.youtube.com/user/gordonramsay showed on pending table
    • Confirmed recurring tip showed on Monthly Contributions panel of brave://rewards as expected
    • Confirmed able to adjust monthly contribution amount from panel
    • Confirmed able to remove monthly contribution from panel and brave://rewards page
  • Custom banner (used prod env)

image
image
image
image
image
image
image

Activity

Encountered #11931, #11932 as an intermittent issue observed where the AC list isn't always as expected for video activity.

Also encountered #11933 for AC list not populating as expected when opening YT videos via right click.

Ran thru steps from brave/brave-core#5440, substituted a shorter video (https://www.youtube.com/watch?v=XoxrQr-w8Kg). Video length was 1min 33s or 93s which matches the duration in the DB.
image
image

Video URL
  • Verified publisher (using staging env):
    • Confirmed tip to https://www.youtube.com/watch?v=uWKCm2GKVyE showed on Tips page of brave://rewards as expected
    • Confirmed recurring tip showed on Monthly Contributions panel of brave://rewards as expected
    • Confirmed able to adjust monthly contribution amount from panel
    • Confirmed able to remove monthly contribution from panel and brave://rewards page
    • Confirmed panel, 1 time tip, and recurring tip banners all had verified checkmark
    • Watched a video for a bit and
      • Confirmed panel showed expected favicon and %
      • Confirmed showed on AC table and % value was as expected
  • Non-verified publisher (using staging env):
    • Confirmed tip to https://www.youtube.com/watch?v=9i4SKHbhbqk showed on pending table
    • Confirmed recurring tip showed on Monthly Contributions panel of brave://rewards as expected
    • Confirmed able to adjust monthly contribution amount from panel
    • Confirmed able to remove monthly contribution from panel and brave://rewards page
    • Watched a video for a bit and
      • Confirmed panel showed expected favicon and %
      • Confirmed showed on AC table and % value was as expected
  • Custom banner (using prod env)

image
image
image
image
image
image
image
image
image
image

Custom URL
  • Verified publisher (using production env):
    • Confirmed tip to https://www.youtube.com/vice showed on Tips page of brave://rewards as expected
    • Confirmed recurring tip showed on Monthly Contributions panel of brave://rewards as expected
    • Confirmed able to adjust monthly contribution amount from panel
    • Confirmed able to remove monthly contribution from panel and brave://rewards page
    • Confirmed panel, 1 time tip, and recurring tip banners all had verified checkmark
  • Non-verified publisher (using production env):
    • Confirmed tip to https://www.youtube.com/ninja showed on pending table
    • Confirmed recurring tip showed on Monthly Contributions panel of brave://rewards as expected
    • Confirmed able to adjust monthly contribution amount from panel
    • Confirmed able to remove monthly contribution from panel and brave://rewards page
  • Custom banner (using prod env) - note, I was unable to find a verified YT publisher with a custom URL and a fully customized banner similar to examples above. However, in the example below the banner text was customized and is different from the default text.

image
image
image

image
image
image
image

History
  • Verified steps from Rewards GreaseLion - YouTube brave-core#5440
  • Confirmed after navigating to various youtube URLs in the same tab the panel displayed the expected publisher info. Used a mix of verified and non-verified publishers
  • Used backward/forward navigation in the same tab and confirmed the panel showed the expected publisher info.
Tab Activation
  • Verified steps from Rewards GreaseLion - YouTube brave-core#5440
  • Opened various YT URLs in multiples tabs and confirmed the panel displayed the expected publisher info. Used a mix of verified and non-verified publishers
  • Switched between the various tabs and confirmed the panel showed the expected publisher info.
Tipping non-YouTube publishers

Regular site:

image

GitHub:
image

Twitch:
image

Encountered #6142

Vimeo (broken):

image

Reddit:
image

Twitter:

image

One time and recurring tips:
image
image

Other Various Tests - Monthly Contributions
  • Set up monthly tips for each of the following types of verified publishers. Confirmed at expected time the monthly contributions were made:
    image
    image
    image
    image
    image
Other Various Tests - panel and banner messaging
  • Confirmed non-verified YT publishers show specific messaging on panel and tipping banners
    image
    image
    image
    image
    image

  • Confirmed verified, non KYC'd YT publishers show different messaging on panel and tipping banners
    image
    image
    image
    image
    image

  • Confirmed verified, KYC'd YT publishers do not show any messaging on panel and tipping banners
    image
    image
    image

Other Various Tests - Settings
  • Modified AC settings to disable "Allow contribution to videos". Viewed some YT videos for several minutes each. Confirmed no YT videos were added to AC table.
  • Re-enabled "Allow contribution to videos" and visited a new YT video. Confirmed the publisher was added to AC list.
  • Confirmed able to exclude a YT publisher from AC via the toggle on the panel
  • Confirmed able to exclude a YT publisher from AC via the AC list on brave://rewards
  • Confirmed able to exclude a YT publisher from AC via the popup list of AC sites from brave://rewards
  • Confirmed able to remove a YT publisher from the exclude list and have the publisher get re-added to AC
Upgrade

Using 1.15.x , set up the following data:

  • Have a few 1 time tips to verified publishers completed (some to YT publishers, some other publisher types also)
  • Have a few pending 1 time tips to non-verified YT publishers
  • Have a few recurring tips to verified publishers completed (some to YT publishers, some other publisher types also)
  • Have AC list populated with a mix of verified and non-verified sites, a mix of publisher types including YT publishers

Upgrade to 1.16.x version with Greaselion YT tipping.

  • Confirm all data is preserved as expected on brave://rewards.
  • Confirm if you navigate to the various publishers and open the panel, the AC % is preserved (if publisher is part of AC list), the recurring tip amount is preserved (if set up). Confirm panel shows expected information (favicon if applicable, publisher name, etc). Confirm custom tip banner is available (if applicable)
  • Confirm if you view another video by a publisher already on AC list the values in the DB (I looked at duration/visits on activity_info) and AC % change appropriately
  • Confirm you can view a YT video by a publisher not already on AC list and it gets added to AC list
  • For an existing recurring tip, confirm you are able to adjust the recurring tip amount for YT publisher from the panel
  • For an existing recurring tip, confirm you are able to delete a recurring tip for YT publisher

@bsclifton bsclifton changed the title [Desktop] Use GreaseLion for YouTube publishers [Desktop] Use Greaselion for YouTube publishers Oct 26, 2020
@rebron rebron changed the title [Desktop] Use Greaselion for YouTube publishers Use Greaselion for YouTube publishers Oct 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment