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

Twitch panel now shows favicon and panel displays correctly for tipping - #177

Open
jasonrsadler wants to merge 1 commit into
masterfrom
twitch-panel-tip-lib
Open

Twitch panel now shows favicon and panel displays correctly for tipping#177
jasonrsadler wants to merge 1 commit into
masterfrom
twitch-panel-tip-lib

Conversation

@jasonrsadler

@jasonrsadler jasonrsadler commented Nov 11, 2018

Copy link
Copy Markdown
Contributor

@jasonrsadler jasonrsadler self-assigned this Nov 11, 2018
@jasonrsadler
jasonrsadler force-pushed the twitch-panel-tip-lib branch 4 times, most recently from 42a0c5d to d1b0776 Compare November 21, 2018 02:42
@jasonrsadler
jasonrsadler force-pushed the twitch-panel-tip-lib branch 12 times, most recently from 1376727 to 3816335 Compare December 2, 2018 18:50
@jasonrsadler jasonrsadler changed the title WIP - Now able to retrieve Twitch favicon Twitch panel now shows favicon and panel displays correctly for tipping Dec 2, 2018
Comment thread src/bat_get_media.cc Outdated
Comment thread src/bat_get_media.cc Outdated
@jasonrsadler
jasonrsadler force-pushed the twitch-panel-tip-lib branch 4 times, most recently from 1960359 to 9350db7 Compare December 6, 2018 17:49
@jasonrsadler
jasonrsadler force-pushed the twitch-panel-tip-lib branch 2 times, most recently from d0046fc to f2c5eff Compare December 6, 2018 19:49
Comment thread src/bat_get_media.cc
Comment thread src/bat_get_media.cc
Comment thread src/bat_get_media.cc
Comment thread src/bat_get_media.cc
Comment thread src/bat_get_media.cc Outdated
Moving values to static

Moved logging statement below empty string check

Implemented evaluating for twitch publisher

Push evaluation to ledger

Expanded naming of blob evaluate members to describe better.
@diracdeltas

Copy link
Copy Markdown

lgtm now (though i haven't functionally tested it)

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants