-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Click on Twitter favicon in tipping banner redirects to an error page #12214
Comments
this is not regression as the same thing happens for me on previous versions |
@NejcZdovc I am getting an empty screen in Release channel 1.15.x |
yes I get empty screen as well. Problem is that link that we get from server doesn't have https in front and because of that it will not open correctly. Checking with publisher team what to do |
It seems reasonable to not display a social media link if the link is not a valid absolute URL. What do you think @NejcZdovc? |
@zenparsing that would work yup. We should also fix it on publisher side |
Verification passed on
Verified passed with
Verified STR from description Reproduced the issue using 1.17.x Confirmed visiting same publisher on 1.18.x did not show the favicons. Confirmed visiting a publisher with valid URLs in their favicons did show the favicons. Clicking on the favicon directed me to the expected URL. Verification passed on
|
Description
Click on twitter favicon in tipping banner redirects to an error page
Steps to Reproduce
Send tip
Actual result:
Click on twitter favicon in tipping banner redirects to an error page
Expected result:
should not show any error message
Reproduces how often:
Always
Brave version (brave://version info)
Version/Channel Information:
Other Additional Information:
Miscellaneous Information:
cc: @brave/legacy_qa @rebron @NejcZdovc
The text was updated successfully, but these errors were encountered: