When editing a Document stored in a Files section, clicking Do not send notifications for this update causes that option to disappear. Screenshots before and after click attached.
| Comment | File | Size | Author |
|---|---|---|---|
| do-not-send_after-click.png | 15.67 KB | thekenshow | |
| do-not-send_before-click.png | 18.98 KB | thekenshow |
Comments
Comment #2
mpotter commentedHmm, are you sure you are using the dev version of oa_notifications? I could not reproduce this one. Tried it on the Add, Edit, and View pages and clicking the Do not send... checkbox didn't disappear on any of those for document pages in a Files section.
Can you tell me which page you were on that saw this? Also, are comments enabled on that page?
Comment #3
mpotter commentedAh ha, got it...
If you are on the Edit form of the document page and comments are disabled, then the Do not send... option seems to disappear. Working on it.
Comment #4
thekenshow commented> are on the Edit form of the document page and comments are disabled
Yes, that’s exactly where I am. I’m going to have to write more detailed essays so you don’t need a ouija board to figure out my situation ;-)
Comment #5
mpotter commentedOK, let's try this! Committed a3bbb64 to oa_notification, so pull the latest dev and try it again.
Comment #6
mpotter commentedComment #7
mpotter commentedFor other people who might be watching: This is probably the most complicated functionality I've implemented in a long time, and it's one of those things that is ridiculously hard to test via automated testing. There are lots of different combinations of 1) node add/edit/view pages, 2) the Override checkbox (ajax), the Textfield for adding things (ajax plus autocomplete), the X buttons for removing (ajax) the Do Not Send checkbox (ajax) and the Expand all (ajax), 3) and the exact order of these different elements that you interact with.
For example "Checking the Do Not Send checkbox After clicking the X button to remove somebody, when on the Node Edit form".
It would probably take me longer to write tests for all of this than it has to write the actual functionality! (Ajax tests are some the hardest to write and debug)
Anyway, I *really* appreciate getting real human testing on this system. It's important to get it right.
Comment #8
thekenshow commentedInstalled, tested, looks good. The checkbox works, stays in place. When I check it and then save, and then edit, it’s restored to unchecked - as I expected.