Closed mathisdt closed 4 years ago
BTW, can you briefly state in the commit message(s) which changes were needed to make this work in TB68?
I'm afraid I can't tell what was needed to support TB68 - as I stated in the title, Samuel and Yves did the coding. I only created this PR to consolidate their changes which I got from the links in their comments on https://blog.qiqitori.com/?p=194.
@qiqitori, probably the use of manifest.json
instead of install.rdf
helps with the compatibility to TB68, but I'm not an extension developer so I don't really know if this is the only necessary change.
Can I help with something else, or are you confident enough with the changes to merge this PR?
@qiqitori I'd like to kindly ask if there is any obstacle to be overcome or if this PR can be merged? As far as I can see, all of your comments are addressed.
Indentation still looks a bit wonky but I'm gonna merge anyway
@mathisdt Merged. BTW -- I don't really remember how to test this extension (I just made it on request for someone). Can you confirm that the code still works and that the default behavior is still the same as the original extension?
First I used the original extension, then this version. Both worked similar, but with this version there are more options which makes it more fun to use. And yes, I can confirm that the default behaviour is the same (changing the sender address automatically for replies).
extensions.replyasoriginalrecipient.patterns
manifest.json
instead ofinstall.rdf
also see the corresponding comments at https://blog.qiqitori.com/?p=194