Skip to content

[WIP] test(core): test UI abort after missing ButtonAck#6863

Closed
romanz wants to merge 5 commits into
romanz/ignore-hostfrom
romanz/2605/test-ignore-host
Closed

[WIP] test(core): test UI abort after missing ButtonAck#6863
romanz wants to merge 5 commits into
romanz/ignore-hostfrom
romanz/2605/test-ignore-host

Conversation

@romanz

@romanz romanz commented May 2, 2026

Copy link
Copy Markdown
Contributor

romanz added 3 commits May 2, 2026 11:36
No more button requests / errors will be sent later during
the backup workflow after an I/O-related error/timeout.

Host will be ignored until the backup workflow is over -
sending the final "Success" response may fail.

Other workflows can be aborted after user cancellation via `_waiting_screen()`.
@coderabbitai

coderabbitai Bot commented May 2, 2026

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 02c8e754-68d4-4afc-9dfa-0f193410c1c3

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch romanz/2605/test-ignore-host

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@romanz romanz added the translations Put this label on a PR to run tests in all languages label May 2, 2026
@trezor-bot trezor-bot Bot added this to Firmware May 2, 2026
@github-project-automation github-project-automation Bot moved this to 🔎 Needs review in Firmware May 2, 2026
@github-actions

github-actions Bot commented May 2, 2026

Copy link
Copy Markdown

en main(all)

model device_test click_test persistence_test
T2T1 test(all) main(all) test(all) main(all) test(all) main(all)
T3B1 test(all) main(all) test(all) main(all) test(all) main(all)
T3T1 test(all) main(all) test(all) main(all) test(all) main(all)
T3W1 test(all) main(all) test(all) main(all) test(all) main(all)
Translations

cs main(all)

model device_test click_test
T2T1 test(all) main(all) test(all) main(all)
T3B1 test(all) main(all) test(all) main(all)
T3T1 test(all) main(all) test(all) main(all)
T3W1 test(all) main(all) test(all) main(all)

de main(all)

model device_test click_test
T2T1 test(all) main(all) test(all) main(all)
T3B1 test(all) main(all) test(all) main(all)
T3T1 test(all) main(all) test(all) main(all)
T3W1 test(all) main(all) test(all) main(all)

es main(all)

model device_test click_test
T2T1 test(all) main(all) test(all) main(all)
T3B1 test(all) main(all) test(all) main(all)
T3T1 test(all) main(all) test(all) main(all)
T3W1 test(all) main(all) test(all) main(all)

fr main(all)

model device_test click_test
T2T1 test(all) main(all) test(all) main(all)
T3B1 test(all) main(all) test(all) main(all)
T3T1 test(all) main(all) test(all) main(all)
T3W1 test(all) main(all) test(all) main(all)

pt main(all)

model device_test click_test
T2T1 test(all) main(all) test(all) main(all)
T3B1 test(all) main(all) test(all) main(all)
T3T1 test(all) main(all) test(all) main(all)
T3W1 test(all) main(all) test(all) main(all)

Latest CI run: 25274090159

@romanz romanz force-pushed the romanz/2605/test-ignore-host branch from 15e376c to 6e94231 Compare May 2, 2026 15:52
@romanz romanz force-pushed the romanz/2605/test-ignore-host branch 2 times, most recently from 4e4e760 to aa59dd4 Compare May 2, 2026 20:31
@romanz romanz force-pushed the romanz/2605/test-ignore-host branch from aa59dd4 to 934a72c Compare May 3, 2026 07:53
@romanz romanz force-pushed the romanz/ignore-host branch from 3458712 to 0043663 Compare May 22, 2026 13:11
@romanz romanz force-pushed the romanz/ignore-host branch 8 times, most recently from 39f2229 to 186a6a6 Compare June 2, 2026 16:20
@romanz

romanz commented Jun 2, 2026

Copy link
Copy Markdown
Contributor Author

Done in #6651

@romanz romanz closed this Jun 2, 2026
@romanz romanz deleted the romanz/2605/test-ignore-host branch June 2, 2026 17:48
@trezor-bot trezor-bot Bot moved this from 🔎 Needs review to ✅ Done (no QA) in Firmware Jun 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

translations Put this label on a PR to run tests in all languages

Projects

Status: ✅ Done (no QA)

Development

Successfully merging this pull request may close these issues.

1 participant