Conversation
dependabot
Bot
added
dependencies
Owner
|
@dependabot rebase |
Owner
|
@dependabot rebase |
dependabot
Bot
force-pushed
the
dependabot/nuget/tests/WebhookEngine.API.Tests/multi-c0501c4ded
branch
from
fd75b32 to
c6aec26
Compare
Contributor
Author
|
The base commit for this pull request has not changed. |
2 similar comments
Contributor
Author
|
The base commit for this pull request has not changed. |
Contributor
Author
|
The base commit for this pull request has not changed. |
--- updated-dependencies: - dependency-name: coverlet.collector dependency-version: 10.0.0 dependency-type: direct:production update-type: version-update:semver-major - dependency-name: coverlet.collector dependency-version: 10.0.0 dependency-type: direct:production update-type: version-update:semver-major - dependency-name: coverlet.collector dependency-version: 10.0.0 dependency-type: direct:production update-type: version-update:semver-major - dependency-name: coverlet.collector dependency-version: 10.0.0 dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
dependabot
Bot
force-pushed
the
dependabot/nuget/tests/WebhookEngine.API.Tests/multi-c0501c4ded
branch
from
c6aec26 to
8174378
Compare
voyvodka
deleted the
dependabot/nuget/tests/WebhookEngine.API.Tests/multi-c0501c4ded
branch
This was referenced May 5, 2026
voyvodka
added a commit
that referenced
this pull request
May 8, 2026
…skeleton, no flash, awaited refetches (#68) Six findings from the dashboard-expert audit, all in one PR before F12. a11y on Modal (audit findings #9 + #10) — the dialog was an unlabelled, focus-leaky panel: no role="dialog", no aria-modal, no aria-labelledby, and Tab/Shift+Tab walked right out of the modal into the document underneath. Fixed all four: - Panel now carries role="dialog" + aria-modal="true" with useId-stable aria-labelledby (and aria-describedby when a description is passed). - Close button got an explicit aria-label="Close dialog". - New keydown handler implements a Tab/Shift+Tab focus trap that wraps focus between the first/last focusable descendants (or pins to the panel itself when nothing inside is focusable). useDeliveryFeed reconnect cleanup (audit finding #19) — onreconnected now resets lastHealthChange to null. Without this, an EndpointsPage that re-subscribed after a disconnect would re-apply the pre-disconnect snapshot from cache, masking whatever the server may have transitioned to during the gap. DeliveryLogPage skeleton (audit finding #22) — replaced the bare "Loading..." text with a layout-preserving skeleton card (header strip + two attempt blocks) plus aria-busy on the wrapper. No layout shift on completion. EndpointsPage flash (audit finding #23) — the page was checking applications.length === 0 before the apps fetch had returned, briefly flashing "Create an application first" on every cold load. New applicationsLoading state gates that branch so the spinner shows until the apps fetch completes; only then does the empty / endpoints branch render. ApplicationsPage floating promise (audit finding #26) — both handleCreate and the delete-confirm onConfirm called fetchApps() without awaiting it, so any refetch rejection became an unhandled promise. Both now await, landing failures inside the surrounding try-catch / setError flow.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Updated coverlet.collector from 8.0.1 to 10.0.0.
Release notes
Sourced from coverlet.collector's releases.
10.0.0
Improvements
--coverlet-file-prefixoption for unique report files #1869Fixed
Maintenance
Diff between 8.0.1 and 10.0.0
Commits viewable in compare view.