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

Attribution for Will Christiansen background image(s) link is broken #8848

Closed
mbacchi opened this issue Mar 25, 2020 · 5 comments · Fixed by brave/brave-core#5068
Closed

Comments

@mbacchi
Copy link
Contributor

mbacchi commented Mar 25, 2020

Test plan

  1. Refresh NTP until photo is The photo Glacier Peak
  2. Click website link in attribution
  3. It should work

Description

The photo Glacier Peak came up and I clicked on the attribution but it brought me to chrome://newtab/www.theskyfolk.com instead of the correct URL.

It looks like the background.ts link field needs to include https:// at the beginning of the link address, else it will prepend chrome://newtab to the beginning, sending users to the wrong location.

Image 14 also would require the complete link, in addition to image 13.

mbacchi added a commit to brave/brave-core that referenced this issue Mar 26, 2020
@rebron rebron added the priority/P3 The next thing for us to work on. It'll ride the trains. label Mar 27, 2020
@mbacchi
Copy link
Contributor Author

mbacchi commented Mar 27, 2020

@kjozwiak I opened a PR (5068) to fix this but the test-install steps failed on mac/windows. I'm going to restart the pr builder and see if it works today.

@kjozwiak
Copy link
Member

Awesome, thanks @mbacchi 👍 Ran into this when I started running through #6904 and noticed you created the issue. Thanks for finding/spotting it!

@rebron @bsclifton should we get this uplifted into 1.7.x as the new images will be on Release in ~a week or so as per #6904?

@bsclifton
Copy link
Member

Sure- that's a small enough one. Let me check out the PR real quick 👍

@LaurenWags
Copy link
Member

LaurenWags commented Mar 30, 2020

Verified passed with

Brave 1.7.78 Chromium: 80.0.3987.149 (Official Build) dev (64-bit)
Revision 5f4eb224680e5d7dca88504586e9fd951840cac6-refs/branch-heads/3987_137@{#16}
OS macOS Version 10.14.6 (Build 18G3020)
  • Verified test plan from description

Screen Shot 2020-03-30 at 9 02 18 AM

  • Also verified for this image from Will Christiansen

Screen Shot 2020-03-30 at 9 01 24 AM

Verification passed on

Brave 1.7.78 Chromium: 80.0.3987.149 (Official Build) dev (64-bit)
Revision 5f4eb224680e5d7dca88504586e9fd951840cac6-refs/branch-heads/3987_137@{#16}
OS Windows 10 OS Version 1803 (Build 17134.1006)
  • Verified test plan from description
    image
    image

Verification passed on

Brave 1.7.78 Chromium: 80.0.3987.149 (Official Build) dev (64-bit)
Revision 5f4eb224680e5d7dca88504586e9fd951840cac6-refs/branch-heads/3987_137@{#16}
OS Ubuntu 18.04 LTS
  • Verified test plan from description

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants