franzenzenhofer / f19n-obtrusive-livetest

A sandboxed, extendable testing chrome extension and framework! It runs pre-defined and custom tests on each page that you visit.
https://chrome.google.com/webstore/detail/f19n-obstrusive-live-test/jbnaibigcohjfefpfocphcjeliohhold?hl=en
55 stars 1 forks source link

adding rules broken #57

Closed franzenzenhofer closed 7 years ago

franzenzenhofer commented 7 years ago

whenever I now edit a default rule, after the edit the default rule vanishes - and the custom rules get forgotten

franzenzenhofer commented 7 years ago

hi, could recreate the

A) "custom rules deleted with default rules update" issue

but not the B) "default rules vanishes after update" issue (but it was there....)

ad A)

create a blank default rule (no text in file) image

add some custom rules

image

update default rule (as some acutal JS)

-> custom rules are gone image

ad B) could not repleciate but please investigate

MVP critical bug

franzenzenhofer commented 7 years ago

hmmm, i just created this default rule and whatever I do now (even a blank install) this rule does not show up in the setting panel (and does not get executed) https://github.com/franzenzenhofer/f19n-livetest-chrome-extension/blob/master/src/public/default-rules/static-idle-head-title-compare.js

neuling commented 7 years ago

oh noe. will look into it now.

neuling commented 7 years ago

removing custom rules on default-rules changes is fixed.

neuling commented 7 years ago

the other problem (not being able to add new rules) should also be fixed. we hit the max quota on the storage. i added the permissions for "unlimitedStorage" to the manifest.

https://developer.chrome.com/extensions/storage#property-local

franzenzenhofer commented 7 years ago

hi @neuling thx

but deinstalled extension built the extension again quit the browser started the browser again added extension again https://github.com/franzenzenhofer/f19n-livetest-chrome-extension/blob/master/src/public/default-rules/static-idle-head-title-compare.js still does not show up?

neuling commented 7 years ago

hmm. strange. will check it again and fix it asap.

neuling commented 7 years ago

sorry @franzenzenhofer. i found two additional bugs. should finally be fixed now. hopefully. let me know if everything works.