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

[Desktop] Use Greaselion for Twitter publishers #11462

Closed
NejcZdovc opened this issue Aug 27, 2020 · 1 comment · Fixed by brave/brave-core#6671
Closed

[Desktop] Use Greaselion for Twitter publishers #11462

NejcZdovc opened this issue Aug 27, 2020 · 1 comment · Fixed by brave/brave-core#6671

Comments

@NejcZdovc
Copy link
Contributor

NejcZdovc commented Aug 27, 2020

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

@emerick emerick self-assigned this Aug 31, 2020
@srirambv srirambv changed the title Use GreaseLion for Twitter publishers [Desktop] Use GreaseLion for Twitter publishers Sep 9, 2020
@NejcZdovc NejcZdovc modified the milestone: Greaselion 1.0.29 Sep 30, 2020
@NejcZdovc NejcZdovc added this to the 1.17.x - Nightly milestone Oct 6, 2020
@LaurenWags
Copy link
Member

LaurenWags commented Oct 13, 2020

Verification completed using builds below

Brave	1.17.28 Chromium: 86.0.4240.75 (Official Build) nightly (x86_64)
Revision	c69c33933bfc72a159aceb4aeca939eb0087416c-refs/branch-heads/4240@{#1149}
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.29
General SS 1
  • Confirmed visiting non-verified publisher showed the generic Twitter favicon in panel
General SS 2
Standard URL

Note, per internal discussions, confirmed that if a custom favicon exists for the tip banners, this custom favicon will be used everywhere (panel, brave://rewards, etc). This is different than 1.16.x so I wanted to call it out.

  • Verified steps from Rewards Greaselion - Twitter brave-core#6671 on staging env
  • For a verified publisher:
    • Confirmed tip to https://www.twitter.com/emerick 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, and BAT logo all had verified checkmark
  • For a non-verified publisher:
    • Confirmed tip to https://www.twitter.com/brave 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
  • Confirmed custom banner displays for 1 time and recurring tips:
Std URL - verified pub Std URL - non verified pub Std URL - UI2 Std URL - UI1 Std URL - Custom banner1 Std URL - Custom banner2
Excluded URL Excluded URL 1 Excluded URL 2 Excluded URL 3 Excluded URL 4 Excluded URL 5
Activity
  • Verified steps from Rewards Greaselion - Twitter brave-core#6671 on staging env (note, confirmed with Emerick that individual Twitter publishers are not added to AC table)
  • Confirmed only twitter.com is in the activity_info db table
Activity 1 Activity 2
History
  • Verified steps from Rewards Greaselion - Twitter brave-core#6671 on prod env
  • Confirmed after navigating to various Twitter 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.
  • Mixed in clicking on individual tweets and opening tip banner from in-line tip button as well
  • Checked above both when logged in to Twitter and not logged in to Twitter
Tab Activation
  • Verified steps from Rewards Greaselion - Twitter brave-core#6671 on prod env
  • Opened various Twitter 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.
  • Mixed in clicking on individual tweets and opening tip banner from in-line tip button as well
  • Checked above both when logged in to Twitter and not logged in to Twitter
Tipping non-Twitter publishers

Regular site:
regular site

YouTube:
YT

GitHub:
GH

Reddit:
Reddit

Twitch:
Twitch

Vimeo(broken):
Vimeo

Recurring and 1 time tips:
Screen Shot 2020-10-13 at 12 30 34 PM

Other Various Tests - Settings
Brave	1.17.29 Chromium: 86.0.4240.80 (Official Build) nightly (x86_64)
Revision	7ed88b53bda45a2d19efb4f8706dd6b6cad0d3af-refs/branch-heads/4240@{#1183}
OS	macOS Version 10.14.6 (Build 18G3020)
Other Various Tests - panel and banner messaging
  • confirm non verified Twitter pubs show specific messaging on banners/panel
nonverified panel nonverified banner 1 nonverified banner 2
  • confirm verified, KYC'd publishers do not show any messing on banners/panel
verified panel verified banner 1 verified banner 2
  • Unable to confirm verified, non KYC'd YT publishers show different messaging on banners/panel due to a publisher not being of this type on staging env
Brave	1.17.34 Chromium: 86.0.4240.99 (Official Build) nightly (x86_64)
Revision	002668237e13d38aabd6d11c2d216dd22b736ff2-refs/branch-heads/4240@{#1229}
OS	macOS Version 10.14.6 (Build 18G3020)
Upgrade
  • Install previous version (used 1.16.61 on Dev), enable rewards
  • have a few 1 time tips to verified publishers (some to twitter, some to other pub types also)
  • have a few pending 1 time tips to non-verified Twitter publishers
  • have a few recurring tips to verified publishers set up (some to Twitter, some to other pub types also)
  • have AC list populated with a mix of verified and non-verified sites
  • Upgrade to 1.17.x (version with Twitter Greaselion tipping)
  • confirm all data is preserved 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).
  • For an existing recurring tip, confirm you are able to adjust the recurring tip amount for Twitter publisher from the panel
  • For an existing recurring tip, confirm you are able to delete a recurring tip for Twitter publisher

Note - due to #4557 verified twitter publishers will still show the generic twitter favicon after upgrade. This will be resolved when #6059 is resolved.

Encountered and logged #12159 which pertains to GH tipping while testing (also applies to new profiles on Nightly as well as upgrades)

Accessing twitter via a deep link

While not logged in to Twitter I found a link to a tweet in an article and clicked on it:

https://www.buzzfeed.com/daniellaemanuel/best-pumpkin-carving?origin=web-hf
Screen Shot 2020-10-15 at 1 14 43 PM

I was directed to Twitter in a new tab as expected. I opened the panel and confirmed the panel showed the expected twitter user's info:
Screen Shot 2020-10-15 at 1 15 39 PM


Verification passed on

Brave 1.17.32 Chromium: 86.0.4240.80 (Official Build) nightly (64-bit)
Revision 7ed88b53bda45a2d19efb4f8706dd6b6cad0d3af-refs/branch-heads/4240@{#1183}
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.29

image

  • Confirmed visiting non-verified publisher showed the generic Twitter favicon in panel

image

Standard URL

Note, per internal discussions, confirmed that if a custom favicon exists for the tip banners, this custom favicon will be used everywhere (panel, brave://rewards, etc). This is different than 1.16.x so I wanted to call it out.

  • Verified steps from Rewards Greaselion - Twitter brave-core#6671 on staging env
  • For a verified publisher:
    • Confirmed tip to https://www.twitter.com/emerick 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, and BAT logo all had verified checkmark
  • For a non-verified publisher:
    • Confirmed tip to https://www.twitter.com/brave 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
  • Confirmed custom banner displays for 1 time and recurring tips:

image
image
image
image
image
image
image

Excluded URL

image
image
image

Activity
  • Verified steps from Rewards Greaselion - Twitter brave-core#6671 on staging env (note, confirmed with Emerick that individual Twitter publishers are not added to AC table)
  • Confirmed only twitter.com is in the activity_info db table

image
image

History
  • Verified steps from Rewards Greaselion - Twitter brave-core#6671 on prod env
  • Confirmed after navigating to various Twitter 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.
  • Mixed in clicking on individual tweets and opening tip banner from in-line tip button as well
  • Checked above both when logged in to Twitter and not logged in to Twitter
Tab Activation
  • Verified steps from Rewards Greaselion - Twitter brave-core#6671 on prod env
  • Opened various Twitter 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.
  • Mixed in clicking on individual tweets and opening tip banner from in-line tip button as well
  • Checked above both when logged in to Twitter and not logged in to Twitter
Tipping non-Twitter publishers

Regular site:
image

YouTube:
image

GitHub:
image

Reddit:
image

Twitch:
image

Vimeo(broken):
image

Recurring and 1 time tips:
image
image

Other Various Tests - Settings
Other Various Tests - panel and banner messaging
  • confirm non verified Twitter pubs show specific messaging on banners/panel
    image
    image
    image

  • confirm verified, KYC'd publishers do not show any messing on banners/panel

image
image
image

Upgrade
  • Install previous version (used 1.16.61 on Dev), enable rewards
  • have a few 1 time tips to verified publishers (some to twitter, some to other pub types also)
  • have a few pending 1 time tips to non-verified Twitter publishers
  • have a few recurring tips to verified publishers set up (some to Twitter, some to other pub types also)
  • have AC list populated with a mix of verified and non-verified sites
  • Upgrade to 1.17.x (version with Twitter Greaselion tipping)
  • confirm all data is preserved 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).
  • For an existing recurring tip, confirm you are able to adjust the recurring tip amount for Twitter publisher from the panel
  • For an existing recurring tip, confirm you are able to delete a recurring tip for Twitter publisher

Note - due to #4557 verified twitter publishers will still show the generic twitter favicon after upgrade. This will be resolved when #6059 is resolved.

Accessing twitter via a deep link

The panel showed the expected twitter user's info:
image


Verification passed on


Brave | 1.17.52 Chromium: 86.0.4240.111 (Official Build) dev (64-bit)
-- | --
Revision | b8c36128a06ebad76af51591bfec980224db5522-refs/branch-heads/4240@{#1290}
OS | Windows 10 OS Version 1903 (Build 18362.1139)


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

  • Confirmed visiting non-verified publisher showed the generic Twitter favicon in panel
    image

Standard URL

Note, per internal discussions, confirmed that if a custom favicon exists for the tip banners, this custom favicon will be used everywhere (panel, brave://rewards, etc). This is different than 1.16.x so I wanted to call it out.

  • Verified steps from Rewards Greaselion - Twitter brave-core#6671 on staging env
  • For a verified publisher:
    • Confirmed tip to https://www.twitter.com/emerick 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, and BAT logo all had verified checkmark
  • For a non-verified publisher:
    • Confirmed tip to https://www.twitter.com/brave 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
    • Confirmed custom banner displays for 1 time and recurring tips:
      image
      image
Excluded URL

image
image
image
image
image

Activity
  • Verified steps from Rewards Greaselion - Twitter brave-core#6671 on staging env (note, confirmed with Emerick that individual Twitter publishers are not added to AC table)
  • Confirmed only twitter.com is in the activity_info db table
    image
    image
History
  • Verified steps from Rewards Greaselion - Twitter brave-core#6671 on prod env
  • Confirmed after navigating to various Twitter 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.
  • Mixed in clicking on individual tweets and opening tip banner from in-line tip button as well
  • Checked above both when logged in to Twitter and not logged in to Twitter
Tab Activation
  • Verified steps from Rewards Greaselion - Twitter brave-core#6671 on prod env
  • Opened various Twitter 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.
  • Mixed in clicking on individual tweets and opening tip banner from in-line tip button as well
  • Checked above both when logged in to Twitter and not logged in to Twitter
Tipping non-Twitter publishers

Regular sites:
image

YouTube:
image

GitHub:
image

Reddit:
image

Twitch:
image
Ensountered ##8462 and ##6851

Vimeo(broken):
image

Recurring and 1 time tips:
image
image

Other Various Tests - Settings
Other Various Tests - panel and banner messaging
  • confirm non verified Twitter pubs show specific messaging on banners/panel
    image
    image
    image

  • confirm verified Twitter pubs show specific messaging on banners/panel
    image
    image
    image

Upgrade
  • Install previous version (used 1.16.61 on Dev), enable rewards
  • have a few 1 time tips to verified publishers (some to twitter, some to other pub types also)
  • have a few pending 1 time tips to non-verified Twitter publishers
  • have a few recurring tips to verified publishers set up (some to Twitter, some to other pub types also)
  • have AC list populated with a mix of verified and non-verified sites
  • Upgrade to 1.17.x (version with Twitter Greaselion tipping)
  • confirm all data is preserved 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).
  • For an existing recurring tip, confirm you are able to adjust the recurring tip amount for Twitter publisher from the panel
  • For an existing recurring tip, confirm you are able to delete a recurring tip for Twitter publisher

Note - due to #4557 verified twitter publishers will still show the generic twitter favicon after upgrade. This will be resolved when #6059 is resolved.

Accessing twitter via a deep link

image

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