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
feat: block black listed content from the app #1361
Conversation
@@ -52,6 +52,10 @@ export const SUPPORT_TRANSACTION_FAILED = 'SUPPORT_TRANSACTION_FAILED'; | |||
// Claims | |||
export const FETCH_FEATURED_CONTENT_STARTED = 'FETCH_FEATURED_CONTENT_STARTED'; | |||
export const FETCH_FEATURED_CONTENT_COMPLETED = 'FETCH_FEATURED_CONTENT_COMPLETED'; | |||
export const FETCH_BLACK_LISTED_CONTENT_STARTED = 'FETCH_BLACK_LISTED_CONTENT_STARTED'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just noting to remind myself: these need to be added to lbry-redux
src/renderer/redux/reducers/app.js
Outdated
@@ -90,7 +120,7 @@ reducers[ACTIONS.AUTO_UPDATE_DOWNLOADED] = state => | |||
}); | |||
|
|||
reducers[ACTIONS.AUTO_UPDATE_DECLINED] = state => { | |||
return Object.assign({}, state, { | |||
Object.assign({}, state, { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You need to return these since there is an opening {
src/renderer/redux/actions/app.js
Outdated
CHECK_BLACK_LISTED_CONTENT_INTERVAL | ||
); | ||
dispatch({ | ||
type: ACTIONS.CHECK_UPGRADE_SUBSCRIBE, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This isn't used anywhere
No description provided.