-
flow
lovetox, code that checks that assumptions are met, and otherwise takes care that they are met, appears to be always sensible
-
jonas’
lovetox, there are good reasons to not have persist items set to true by default, but you should get away with assertions during item publish
-
jonas’
(XEP-0060 assertions, not `assert node.config == correct`)
-
flow
jonas’, xep60 allows to publish with an item and given configuration options that need to be fulfilled? or what are xep60 assertions (there is one hit for 'assert' in xep60)?
-
jonas’
exactly that
-
jonas’
<publish-options/>
-
jonas’
the publish fails with <conflict/> if an option is set to a different value, and the options are applied otherwise
-
flow
ahh that, I stopped reading at 7.1.3
-
lovetox
jonas’, yes i know about publish options
-
lovetox
it about what to do if there is a conflict
-
lovetox
tell the user to contact the admin, or simply force all nodes to my likings✎ -
lovetox
tell the user to contact the admin, or simply force all nodes to my will ✏
-
lovetox
the later is of course much more work and even more error cases to handle
-
jonas’
lovetox, if there’s a conflict, it may not be the admins fault at all
-
jonas’
might also be the user’s fault
-
jonas’
or another client’s fault
-
lovetox
how does that help me :)
-
lovetox
there is not many config options we can change at all, i think access model and persist, and then it ends
-
lovetox
so persist items is the most likely one to fail