Skip to content

Allow skip it too slow to fetch crash report - #449

Merged
kylebonnici merged 4 commits into
split/baseline-pollingfrom
cloud-first-skip-crash-report
Aug 20, 2026
Merged

Allow skip it too slow to fetch crash report#449
kylebonnici merged 4 commits into
split/baseline-pollingfrom
cloud-first-skip-crash-report

Conversation

@kylebonnici

Copy link
Copy Markdown
Contributor

No description provided.

@kylebonnici kylebonnici added doc not required All PRs either need "doc required" or "doc not required". ui not required All PRs either need "ui required" or "ui not required". labels Aug 19, 2026
@kylebonnici
kylebonnici force-pushed the cloud-first-skip-crash-report branch from bd37978 to 0ac8e37 Compare August 19, 2026 20:04

@formalmia formalmia left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the stopwatch handling doesn't handle all cases and currently allows us to get into a bad state (as described in my comment).

I think alternatively we can also use a timeout, which seems more suited to this (we do not use the actual passing of time for anything). Though practically it wouldn't make much difference (apart from maybe saving us about 3 lines of code). I'd personally prefer it though.

Comment thread src/features/flows/nRF54L15_cloud/evaluate/TestCrash.tsx Outdated
Comment thread src/features/flows/nRF54L15_cloud/evaluate/TestCrash.tsx
Comment thread src/features/flows/nRF54L15_cloud/evaluate/TestCrash.tsx
Comment thread src/features/flows/nRF54L15_cloud/evaluate/TestCrash.tsx
Comment thread src/features/flows/nRF54L15_cloud/evaluate/TestCrash.tsx
Comment thread src/features/flows/nRF54L15_cloud/evaluate/TestCrash.tsx Outdated
@kylebonnici
kylebonnici force-pushed the split/baseline-polling branch from de5f650 to 3c2ec96 Compare August 20, 2026 12:59
@kylebonnici
kylebonnici force-pushed the cloud-first-skip-crash-report branch from af4c6e0 to 4388de6 Compare August 20, 2026 12:59
@kylebonnici
kylebonnici requested a review from formalmia August 20, 2026 13:08
kylebonnici and others added 3 commits August 20, 2026 15:14
Co-authored-by: Mia Rosie Lleorenig <279646534+formalmia@users.noreply.github.com>
@kylebonnici
kylebonnici force-pushed the split/baseline-polling branch from 3c2ec96 to 9375234 Compare August 20, 2026 13:14
@kylebonnici
kylebonnici force-pushed the cloud-first-skip-crash-report branch from cd0614a to 8a0dfc3 Compare August 20, 2026 13:14
* Set bt device name

* Feeback

* Feeback
@kylebonnici
kylebonnici merged commit 072cb1f into split/baseline-polling Aug 20, 2026
3 checks passed
@kylebonnici
kylebonnici deleted the cloud-first-skip-crash-report branch August 20, 2026 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc not required All PRs either need "doc required" or "doc not required". ui not required All PRs either need "ui required" or "ui not required".

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants