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

Temporarily remove insertions #3211

Merged
merged 1 commit into from
Aug 17, 2016
Merged

Temporarily remove insertions #3211

merged 1 commit into from
Aug 17, 2016

Conversation

bbondy
Copy link
Member

@bbondy bbondy commented Aug 16, 2016

To be investigated later probably without the use of ad-div info

The branch for ad replacements will be maintained here:
https://github.com/brave/browser-laptop/tree/adReplacements

Auditors: @bridiver

To be investigated later probably without the use of ad-div info

The branch for ad replacements will be maintained here:
https://github.com/brave/browser-laptop/tree/adReplacements

Auditors: @bridiver
@bsclifton bsclifton changed the title Tempoarily remove insertions Temporarily remove insertions Aug 16, 2016
@bbondy bbondy added this to the 0.11.6dev milestone Aug 16, 2016
@bbondy
Copy link
Member Author

bbondy commented Aug 16, 2016

CC @alexwykoff
Please test upgrade case both with global settings and site specific setting.

@bridiver
Copy link
Collaborator

my only concern was upgrade path and sounds like you're already looking into that, so ++

@bbondy bbondy merged commit 8eb0ab8 into master Aug 17, 2016
@bbondy
Copy link
Member Author

bbondy commented Aug 17, 2016

upgrade path already worked for me but will let @alexwykoff confirm.

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.

4 participants