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

広告のフィルタリングを修正 #4

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

domekomi
Copy link

@domekomi domekomi commented Jul 3, 2018

「宣伝をした」のボタンが機能しない問題を修正。
ChromeウェブストアのレビューのまじんGさんよりコードをいただきました。

「宣伝をした」のボタンが機能しない問題を修正。
ChromeウェブストアのレビューのまじんGさんよりコードをいただきました。
@tikusieg
Copy link

tikusieg commented Jul 4, 2018

このコードだと「生放送を広告した」が消せないので139行目の

&& linkTarget.pathname.indexOf("/video/")===0

を削除することで生放送の広告も非表示にすることができます。
もしくは生放送の設定を追加するとか

@tikusieg
Copy link

tikusieg commented Jul 6, 2018

2018/07/05より試験的に始まったコミュニティを経由しない「ユーザー独自の生放送」のニコレポも追加されました。
これのせいで一部スマホアプリにも影響が出ているのですが…
コミットしたいので権限開けていただけると嬉しいです。
それか49行目下に以下を追加しました。
contentsAdd({contentId:"user-nama-start" ,buttonLabel:"生放送を開始" ,userType:1,normalizeText:"[LINK USER]さんが 生放送を開始しました。"});

@domekomi
Copy link
Author

domekomi commented Jul 6, 2018

このPull request(PR)は広告に関するものなので、別のPRにて修正すべきかと思います。
PRは権限がなくても可能です。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants