fix(empty-cards): make undoable, correct result, use backend strings

[david-allison]

Purpose / Description

• Notifies the subscriber after "Check - Empty Cards" removed cards #16945

Fixes

• Fixes Notifies the subscriber after "Check - Empty Cards" removed cards #16945

Approach

• Extract operation to ViewModel
• Extract output to Flow
• add opChanges { }
• Use the result of opChangesWithCount
• Use TR rather than strings
• Add Undo to the snackbar, and only show it for a short duration
• Test!

How Has This Been Tested?

⚠️ Unit tests only

Checklist

• You have a descriptive commit message with a short title (first line, max 50 chars).
• You have commented your code, particularly in hard-to-understand areas
• You have performed a self-review of your own code
• UI changes: include screenshots of all affected screens (in particular showing any new or changed strings)
• UI Changes: You have tested your change using the Google Accessibility Scanner

[github-actions]

Important

Maintainers: This PR contains Strings changes

1. Sync Translations before merging this PR and wait for the action to complete
2. Review and merge the auto-generated PR in order to sync all user-submitted translations
3. Sync Translations again and merge the PR so the huge automated string changes caused by merging this PR are by themselves and easy to review

[mikehardy]

last commit looks good to me, just bikeshedded you on a name.
approving because that naming is not that important to me vs moving right along here, do what you wish

[mikehardy]

you're probably aware but unit tests failed, marked author reply

[david-allison]

you're probably aware but unit tests failed, marked author reply

I wasn't. Fixed!

[lukstbit]

LGTM, thanks!

[github-actions]

Maintainers: Please Sync Translations to produce a commit with only the automated changes from this PR.

Read more about updating strings on the wiki,

• localization-administration
• download-localized-strings