-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Support publisher branded banner #1923
Comments
Verification passed on
Used STR from OP. Test staging server. Verification PASSED on
Verified passed with
|
Updated the banner for |
Hi @NejcZdovc we recently added a new feature to the editor which allows publishers to select tip amounts from a list of presets. @kjozwiak had some reports of the tips not persisting to the client-side, I believe we are serving them correctly from our My guess is that there was some code added to restrict the values to [1,5,10] when Product team decided that a few weeks ago. I think the solution would be to open up the restriction to the following amounts { 1, 5, 10 } |
Description
Currently we are not displaying images that are returned from the server for publishers in site banner.
Steps to Reproduce
Actual result:
Site banner shows default banner
Expected result:
Site banner should show server image if that image exists
Reproduces how often:
Brave version (brave://version info)
Reproducible on current release:
Website problems only:
Additional Information
Currently it's disabled here https://github.com/brave/brave-core/blob/master/components/brave_rewards/resources/donate/components/siteBanner.tsx#L116
We already get all necessary data from the server.
The text was updated successfully, but these errors were encountered: