Fixes - MIFOSAC-498

Changes made in the PR:

• Refactored dialog visibility logic for creating client identifiers:

  • The previous implementation managed the show/hide state of the create identifier dialog directly in the UI layer, leading to inconsistent UI behavior. The logic was moved into the ViewModel, ensuring dialog state is now driven by a single source of truth and tied to UI state updates.

• Standardized success/error feedback: show toast on successful identifier creation and error UI on failures, all driven by a single sealed UI state and SharedFlow events.

• Issue: Delete Identifier flow showed Loading twice;

  • Fix: Ensure a single collection per delete action and emit Loading once before the repository call; removed redundant state updates that caused duplicate emissions.
    Screenshots

• Issue: Pull-to-refresh didn’t trigger loading on main screen; data refreshed silently.

  • Fix: Updated refreshIdentifiers in the ViewModel to explicitly set/loading/reset refresh state and emit a Loading UI state; UI now binds rememberPullRefreshState to the ViewModel’s refreshing flag.

Testing notes

• Create: Open Create Identifier, submit valid data → dialog dismisses, screen enters loading, success toast shows, list refreshes.
• Delete: Delete an identifier → only one loading transition, no duplicate spinners.
• Pull-to-refresh: Swipe down → screen enters loading, refresh indicator shows, content updates, indicator stops.

Before:

After:

Please make sure these boxes are checked before submitting your pull request - thanks!

• Run the static analysis check ./gradlew check or ci-prepush.sh to make sure you didn't break anything

• If you have multiple commits please combine them into one commit by squashing them.