Can you put the screenshot in the Summary section above, and list the dependencies there too? Also put dependency information there, in the following form:
Depends on D8243
Depends on D8332
This special syntax marks them as dependencies.
the UDI comment needs to be investigated, otherwise lgtm
the changed list of remote urls that is repeated below could be put into another helper function
shouldn't this always be called? i.e. when the bookmark is changed to a different UDI, don't we need to update here, even when we had a valid device UDI before? I think this also means this path isn't unit tested yet
Changed the code to always check for udi changes.