feat: telemetry consent, webhook delivery log panel, responsive table…#784
Merged
Conversation
|
@arandomogg Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits. You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀 |
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
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.
Summary
<tr>wrapper inTransactionsTableso rows render correctly on all screen widths ([UI/UX & Dashboard Enhancements] Mobile-optimized Responsive Tables #756)prepareOfflineTransactionbuilds and persists signed XDRs to storage while offline;FluidQueuecan flush them on reconnection ([Architecture & Database] Support Offline Transaction Preparation #716)Changes by issue
#757 — Anonymous Usage & Telemetry Collection Consent
TelemetryConsentSettingscomponent,telemetry-consent.tsutility, andtelemetry-consent.test.tswere already authored on this branch; this commit wires up the missing<div>wrapper indashboard/page.tsxso the consent section renders inside a correctly structured layout.#745 — Webhook Delivery Log History Panel
WebhookDeliveryLog.tsx: fixedCopyButton text=→value=prop bug; removed non-existentvariantprop fromStatusBadge; replaced the global payload footer with a per-row expandable drawer that shows request payload, response headers, and event payload side-by-side in a three-column grid.StatusBadgeto acceptWebhookDeliveryStatus(adds the"retrying"tone mapped to amber with pulse animation).requestPayloadandresponseHeadersfields to all sample records inwebhook-delivery-logs-data.tsto satisfy the type definition.lib/webhook-delivery-logs-data.test.ts(7 unit tests covering pagination, field presence, status coverage, and edge cases); all 11 admin-dashboard unit tests pass.#756 — Mobile-optimised Responsive Tables
<tr className="align-top">opening tag insideTransactionsTable'sFragmentRowmap — without it the browser was forced to infer table structure, breaking cell alignment and the mobile disclosure expansion on narrow viewports.md:hidden,lg:table-cell,xl:table-cell) and expand into card-style<dl>rows on screens below 768 px via theMobileDisclosurepattern.#716 — Support Offline Transaction Preparation
client/src/offline-tx.ts: pure, storage-adapter–based module withprepareOfflineTransaction,getPendingOfflineTransactions,removePreparedTransaction,clearPreparedTransactions,getPendingCount, andvalidatePreparedTransaction. No network access is required; XDRs are serialised to the provided storage key and can be submitted later viaFluidQueue.add.client/src/index.ts.client/src/testUtils/__tests__/offline-tx.test.ts(12 unit tests); all pass.Test plan
npm run test:unitinadmin-dashboard/— 11 tests passnpx vitest run src/testUtils/__tests__/offline-tx.test.tsinclient/— 12 tests pass/admin/dashboard— confirm telemetry toggle renders and layout links are correctly spaced/admin/webhooks/logs— click "Details" on any row; confirm request payload, response headers, and event payload appear in the drawercloses #757
closes #745
closes #756
closes #716