Skip to content
This repository was archived by the owner on Feb 16, 2026. It is now read-only.

[IMP] make picking state syncing configurable#7

Closed
thomaspaulb wants to merge 1 commit into14.0from
14.0-done-state-no-tests
Closed

[IMP] make picking state syncing configurable#7
thomaspaulb wants to merge 1 commit into14.0from
14.0-done-state-no-tests

Conversation

@thomaspaulb
Copy link
Member

By default picking state is forced to Done when SO picking is done, but if picking sync fails, eg if there is a mismatch in SO and PO move lines, user can't correct anything since the picking is already set to Done.

This may be acceptable in cases where 100% of the syncs go OK but we make the setting configurable to support situations where manual corrections are needed.

@thomaspaulb thomaspaulb force-pushed the 14.0-done-state-no-tests branch 2 times, most recently from 2f69a95 to de8d894 Compare September 10, 2024 13:54
By default picking state is forced to Done when SO picking is done, but if picking
sync fails, eg if there is a mismatch in SO and PO move lines, user can't correct
anything since the picking is already set to Done.

This may be acceptable in cases where 100% of the syncs go OK but we make the
setting configurable to support situations where manual corrections are needed.
@thomaspaulb thomaspaulb force-pushed the 14.0-done-state-no-tests branch from de8d894 to 9b20b20 Compare September 10, 2024 14:01
@thomaspaulb
Copy link
Member Author

Integrated within #8

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Comments