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

In "Preferences" page, the "Clear Rules" button of blacklist doesn't work #102

Open
alfred-zhong opened this issue Feb 8, 2017 · 3 comments
Labels

Comments

@alfred-zhong
Copy link

alfred-zhong commented Feb 8, 2017

As I said in the title.
I added several rules in the blacklist and clicked the button. After seconds the plugin restarted and I open the "Preferences" page again. The rules is still there.
Here are the code:

function clearRules() {
    chrome.storage.local.get(['blacklist'], function(items) {
        var blacklist = items['blacklist'];
        blacklist['SITE'] = ['chrome-ui://newtab']
        chrome.storage.local.set({'blacklist':blacklist});
    });
    notie.alert(1, 'Deleted Rules. Restarting WorldBrain...', 2)
    setTimeout(function() {
        chrome.runtime.reload()
    }, 2000);
}

I thought it doesn't work. Right?


Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@blackforestboi
Copy link
Member

Hello Alfred!

Thanks for notifying us about this issue.

Yes indeed it doesn't work. I just tested it.
We are currently in a refactoring of the code and afterwards this bug should be gone as well.
Sorry for the inconvenience.

What does work though is to remove them separately via the delete button and then press "save"
(Except for the standard ones, which will pop up again - will also be removed with the new update)

Hope that helps!
Oliver

@wimpykid26
Copy link

@oliversauter Hello, I would like to work on this if you let me. Just a small question?. Clear rules deletes each and every existing rule right from the 'blacklist'?

wimpykid26 added a commit to wimpykid26/Research-Engine that referenced this issue Mar 1, 2017
The clear rules button was not working.
This patch deletes the existing rules.

Fixes WorldBrain#102
wimpykid26 added a commit to wimpykid26/Research-Engine that referenced this issue Mar 1, 2017
The Clear rules button was not working.
This patch clears the existing blacklist,
except the default ones.

fixes WorldBrain#102
@blackforestboi
Copy link
Member

Hey @wimpykid26

first of all, thanks a lot for your proactivity and offering your support.

To your question: yes, just delete them all.

Although workflow wise it is to notice, that we are currently in the mode of refactoring and porting all the features the (Re)search-Engine has to the WebMemex code base.

There we don't have the blacklist yet, but are in the process of building it.
Its a little bit bigger task than deleting all the entries, but If you feel comfortable doing that, it would help a lot to push the project further (and making it available for Firefox users).
It's basically the "backend" part of what @aquibm is working on here: WebMemex/webmemex-extension#20

How does that sound?

If good, then it is a good next step to loop in @Treora for the technical specifics.

Thanks again and a great rest of the week!

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

No branches or pull requests

3 participants