970319 - Adding / removing dynamically rules to Page-Mod's `include` should affect existing documents

Status: RESOLVED WONTFIX

(Add-on SDK Graveyard :: General, defect)

Description

[Matteo Ferretti [:zer0] [:matteo]]

If `attachTo` has "existing" has value, add or remove rules to `include` property should affect existing documents and not only be applied to new documents.

Comment 1

[Irakli Gozalishvili [:irakli] [:gozala] [@gozala]]

We discussed this issue at the triage and decided that it's probably better to just disallow making changes to `include` after worker is created rather than keeping it's changes in sync with existing documents. It's easy enough to create new mod with a modified filter.

Given that we allow mutations now, we should just deprecate that by printing a warning & also pointing out that changes won't reflect on existing documents & in a future they'll be ignored entirely.

Comment 2

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

WONTFIXing per comment 1