Fix SelectEditor to ensure new value is flushed before editing stops (#…

…3748)

CHANGELOG.md

@@ -20,6 +20,10 @@
     * New `XH.genUUID()` method to generate UUIDs for use as Correlation IDs or elsewhere.
 * `GridModel` will now accept `false` as a value to omit context menus.
 
+### 🐞 Bug Fixes
+
+* Fixed `SelectEditor` to ensure new value is flushed before editing stops.
+
 ### ⚙️ Technical
 
 * Remove context menus from column choosers.

desktop/cmp/grid/editors/SelectEditor.ts

@@ -7,6 +7,7 @@
 import {hoistCmp} from '@xh/hoist/core';
 import {select, SelectProps} from '@xh/hoist/desktop/cmp/input';
 import '@xh/hoist/desktop/register';
+import {wait} from '@xh/hoist/promise';
 import {EditorProps} from './EditorProps';
 import './Editors.scss';
 import {useInlineEditorModel} from './impl/InlineEditorModel';
@@ -26,7 +27,9 @@ export const [SelectEditor, selectEditor] = hoistCmp.withFactory<SelectEditorPro
                 hideDropdownIndicator: true,
                 hideSelectedOptionCheck: true,
                 selectOnFocus: false,
-                onCommit: flushOnCommit ? () => props.agParams.stopEditing() : null,
+                onCommit: flushOnCommit
+                    ? () => wait().then(() => props.agParams.stopEditing())
+                    : null,
                 rsOptions: {
                     styles: {
                         menu: styles => ({