Skip to content
This repository has been archived by the owner on Dec 12, 2018. It is now read-only.

Youtube media publishers now show properly in rewards panel #151

Merged
merged 10 commits into from
Oct 29, 2018

Conversation

jasonrsadler
Copy link
Contributor

@jasonrsadler jasonrsadler commented Oct 19, 2018

Fixes #116

core implementation brave/brave-core#699

.gitignore Outdated
.idea
.vscode
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did you mean to add this?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, .vscode directory should not be checked in.

@NejcZdovc NejcZdovc removed their request for review October 24, 2018 09:05
@jasonrsadler jasonrsadler changed the title Youtube and Twitch media publishers now show properly in rewards panel Youtube media publishers now show properly in rewards panel Oct 25, 2018
@jasonrsadler jasonrsadler force-pushed the pub_media_panel branch 3 times, most recently from d9533e2 to d087eef Compare October 25, 2018 17:53
@jasonrsadler jasonrsadler reopened this Oct 25, 2018
@jasonrsadler jasonrsadler force-pushed the pub_media_panel branch 2 times, most recently from 7c6585c to e5a4f30 Compare October 25, 2018 21:27
@NejcZdovc NejcZdovc force-pushed the pub_media_panel branch 4 times, most recently from 484a1df to a34bdd0 Compare October 29, 2018 09:56
@NejcZdovc NejcZdovc self-assigned this Oct 29, 2018
@NejcZdovc NejcZdovc force-pushed the pub_media_panel branch 2 times, most recently from 550ae3a to fd0cac3 Compare October 29, 2018 09:58
Jason Sadler and others added 10 commits October 29, 2018 13:24
…rewards panel

Fixed getting favicon for Twitch vod

Update for youtube showing up right away in panel.

Stabilized youtube media publisher panel

Removing debug log lines

Cleaned up redundant database functionality

Stabilized youtube media publisher panel

clean up debug comments

Replaced accidentally removed line

Merge

Fixes exclude/include on fresh site
Copy link
Contributor Author

@jasonrsadler jasonrsadler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

++ on your part

@jasonrsadler
Copy link
Contributor Author

I am seeing some problems with certain custom paths for some publishers not showing up. Created an issue in brave/brave-browser#1886

@NejcZdovc NejcZdovc merged commit e5e7586 into master Oct 29, 2018
@NejcZdovc NejcZdovc deleted the pub_media_panel branch October 29, 2018 20:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants