Changes between Initial Version and Version 1 of Ticket #23305, comment 28


Ignore:
Timestamp:
2024-02-14T16:31:37+01:00 (9 months ago)
Author:
GerdP

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #23305, comment 28

    initial v1  
    66I think the new method `getResolvableKeys()` is straightforward but it somehow bypasses all the logic in the conflict resolver code.
    77
    8 I've noticed that there is special code to handle tags like `step_count` or `capacity`. I'll have to verify that this patch doesn't break anything here.
     8I've noticed that there is special code to handle tags like `step_count` or `capacity`. I'll have to verify that this patch doesn't break anything here. Edit: Works as before.
     9
     10Possible problem: Non-expert users may have decided to suppress the dialogs with decision cancel, so that combine ways silently refuses to work when there is any conflict. Now they will see the complex conflict dialog instead. So maybe better change the patch to keep the non-expert dialogs?