fix(amas): friendlier import-bar text when count not yet known

Replace the alarming "Reconnecting to import in progress..." fallback with
"Importing employees, please wait...". It showed whenever the bar had no
count yet — briefly before the first tick, and for the whole run when the
server returns no progress (e.g. a worker still on old code). The wording
made a working import look broken.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This commit is contained in:
Ali 2026-06-15 12:50:40 +00:00
parent b911c6a1ae
commit 110a68f8b4
1 changed files with 4 additions and 2 deletions

View File

@ -724,7 +724,7 @@ function attach_amas_import_listeners(listview, asan_login_name, total_hint) {
__('Importing from ƏMAS'), 0, total, __('Importing from ƏMAS'), 0, total,
total total
? __('Starting import of {0} employees...', [total]) ? __('Starting import of {0} employees...', [total])
: __('Reconnecting to import in progress... waiting for next update') : __('Importing employees, please wait...')
); );
ensure_cancel_button(dialog); ensure_cancel_button(dialog);
@ -743,7 +743,9 @@ function attach_amas_import_listeners(listview, asan_login_name, total_hint) {
} else if (knownTotal > 0) { } else if (knownTotal > 0) {
detail = verb + __('{0} of {1}', [lastCurrent, knownTotal]); detail = verb + __('{0} of {1}', [lastCurrent, knownTotal]);
} else { } else {
detail = __('Reconnecting to import in progress... waiting for next update'); // Running, but no count yet (first tick not in, or server hasn't
// populated progress). Keep it reassuring rather than alarming.
detail = __('Importing employees, please wait...');
} }
// Guard against total=0 (NaN width); show_progress reuses the dialog. // Guard against total=0 (NaN width); show_progress reuses the dialog.
const d = frappe.show_progress( const d = frappe.show_progress(