Problem/Motivation
By request, I'm testing 8.x-2.3-rc1.
In #2969418: Allow a separate filter format for plain text vs. HTML emails we introduced forward_update_8201() to split the forward filter setting into an HTML vs. plain text setting.
Unfortunately, that update leaves the original setting still in config:
forward_filter_format: email_html
Since that's not defined in the schema, we get schema validation errors when you run tests after upgrading.
It's also leaking stuff into config we're no longer using, leading to possible confusion and human error.
Proposed resolution
Add a new update to purge the dead setting from config and re-save.
Remaining tasks
Do it.- Test.
- Commit.
User interface changes
None.
API changes
None.
Data model changes
None, really. Fixes a mismatch between actual config and expected config schema.
Release notes snippet
Probably none. TBD.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | 3089276-4.patch | 1.06 KB | dww |
Comments
Comment #2
dwwComment #3
dwwOh yeah, we should remove it from the default forward.settings.yml we ship, too. ;)
Comment #4
dwwWhoops, sorry, my local copy where I rolled the patch was accidentally a 8.x-3.x checkout. This should work better for the 8.x-2.x branch.
Comment #6
john.oltman commentedComment #7
dwwThanks, and sorry for introducing this bug in the first place. ;)
Cheers,
-Derek