fix(ui): include identity name in auth key error message#746
fix(ui): include identity name in auth key error message#746thepastaclaw wants to merge 1 commit into
Conversation
When an identity lacks an authentication key for signing document transitions, the error banner now includes the identity alias or ID so users can identify which of their identities has the issue. Closes dashpay#743
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Superseded by #750 which includes this fix plus suppresses the error on startup auto-select. |
Summary
When an identity lacks an authentication key for signing document transitions, the error banner now includes the identity alias (or base58 ID if no alias) so users can tell which identity has the issue.
Before:
Identity doesn't have an authentication key for signing document transitionsAfter:
Identity 'MyEvonode' doesn't have an authentication key for signing document transitionsThis is especially helpful for users with many identities (the reporter has 16).
Closes #743
Changes
src/ui/identities/mod.rs: Useformat!()withdisplay_string()instead of static stringsrc/backend_task/identity/register_dpns_name.rs: Same — also switched from.ok_or()to.ok_or_else()to avoid eager allocation