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

Missing images in brave:rewards #1869

Closed
btlechowski opened this issue Oct 26, 2018 · 7 comments
Closed

Missing images in brave:rewards #1869

btlechowski opened this issue Oct 26, 2018 · 7 comments

Comments

@btlechowski
Copy link

Steps to Reproduce

  1. Clean install
  2. Go to brave:rewards

Actual result:

Images for Ads, Auto-Contribute, Tips are missing
image
image

Expected result:

image

Reproduces how often:

always

Brave version (brave://version info)

Brave 0.55.21 Chromium: 70.0.3538.77 (Official Build) (64-bit)
Revision 0f6ce0b0cd63a12cb4eccea3637b1bc9a29148d9-refs/branch-heads/3538@{#1039}
OS Windows 7 x64

Reproduced on Win 10 x64 by @kjozwiak
Not reproducible on Linux and Mac (checked by @kjozwiak and @srirambv )

Reproducible on current release:

no

cc @NejcZdovc @GeetaSarvadnya @LaurenWags

@NejcZdovc
Copy link
Contributor

cc @bridiver I think that we need to pull your PR to 55 as well, because this was not a problem before

@bbondy
Copy link
Member

bbondy commented Oct 27, 2018

@NejcZdovc What introduced it? We should consider kicking out what introduced it instead of pulling in more things which might or might not fix it

@NejcZdovc
Copy link
Contributor

the only thing that I can think of is big refactor move brave/brave-core#719

Sadly I think we can't back it out. It's just to big and stuff already landed after it

@NejcZdovc
Copy link
Contributor

this only happens on release, can't reproduce on 0.55.x branch directly

@kjozwiak
Copy link
Member

this only happens on release, can't reproduce on 0.55.x branch directly

Getting the same results as @NejcZdovc, using Win 10 x64, went through the following:

  • 0.57.3 Chromium: 70.0.3538.67 (Official Build) dev (64-bit) - couldn't reproduce
  • 0.56.6 Chromium: 70.0.3538.67 (Official Build) beta (64-bit) - couldn't reproduce
  • 0.55.21 Chromium: 70.0.3538.77 (Official Build) (64-bit) - reproduced

@NejcZdovc
Copy link
Contributor

closing as this is not a problem in 56

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Nov 1, 2018

Verification Passed on

Brave 0.56.7 Chromium: 70.0.3538.77 (Official Build) beta (64-bit)
Revision 0f6ce0b0cd63a12cb4eccea3637b1bc9a29148d9-refs/branch-heads/3538@{#1039}
OS Windows

Went through verification using the following build under Win 10 x64 - PASSED

Brave 0.56.7 Chromium: 70.0.3538.77 (Official Build) beta (64-bit)
Revision 0f6ce0b0cd63a12cb4eccea3637b1bc9a29148d9-refs/branch-heads/3538@{#1039}
OS Windows

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