Skip to content

Add retry actions for platform webhook deliveries#2519

Open
everettbu wants to merge 2 commits into
masterfrom
dnywh/feat/webhook-delivery-retry
Open

Add retry actions for platform webhook deliveries#2519
everettbu wants to merge 2 commits into
masterfrom
dnywh/feat/webhook-delivery-retry

Conversation

@everettbu

Copy link
Copy Markdown

Mirror of supabase/supabase#43846
Original author: dnywh


What kind of change does this PR introduce?

Feature / UX update.

What is the current behavior?

Platform webhook deliveries cannot be retried from the deliveries table or the delivery details sheet.

What is the new behavior?

  • Adds a retry action to individual platform webhook deliveries in the endpoint deliveries table
  • Adds a retry action to the shared delivery details sheet footer
  • Limits retry to non-success deliveries
  • Applies the table action hit area guidance from the design system

Successful deliveries should also not expose retry in this mock flow.

Additional context

  • Addresses DEPR-340.
  • In the current mock implementation, queueing updates the existing delivery in place: it sets the status to pending, clears responseCode, and refreshes attemptAt to the current timestamp.
  • The mock does not create a separate queued job or append a new delivery record yet; the same delivery is resurfaced at the top of the list because deliveries are sorted by latest attempt time.

@github-actions

Copy link
Copy Markdown
Contributor

🎭 Playwright Test Results

passed  157 passed
skipped  5 skipped

Details

stats  162 tests across 20 suites
duration  2 minutes, 50 seconds
commit  d772a60

Skipped tests

Features › assistant.spec.ts › AI Assistant › Can send a message to the assistant and receive a response
Features › sql-editor.spec.ts › SQL Editor › snippet favourite works as expected
Features › sql-editor.spec.ts › SQL Editor › share with team works as expected
Features › sql-editor.spec.ts › SQL Editor › folders works as expected
Features › sql-editor.spec.ts › SQL Editor › other SQL snippets actions work as expected

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants