Skip to content

fix(shared): close squad share modal after external link post#5978

Merged
rebelchris merged 2 commits intomainfrom
eng-1358-feedback-feature-request-modal-does-not-close-after-posting
May 4, 2026
Merged

fix(shared): close squad share modal after external link post#5978
rebelchris merged 2 commits intomainfrom
eng-1358-feedback-feature-request-modal-does-not-close-after-posting

Conversation

@rebelchris
Copy link
Copy Markdown
Contributor

@rebelchris rebelchris commented May 4, 2026

Summary

  • close the squad share modal after successful external-link submissions
  • keep existing modal close behavior for the other successful post flows
  • add focused regression tests for the shared hook and modal wiring

Key Decisions

  • introduced a dedicated onComplete callback in usePostToSquad so the external-link path can signal success without fabricating a Post object
  • kept onPostSuccess reserved for flows that actually return a Post
  • narrowed CreateSharedPostModal back to lazy-modal props to keep the close path simple

Closes ENG-1358


Created by Huginn 🐦‍⬛

Preview domain

https://eng-1358-feedback-feature-reques.preview.app.daily.dev

@vercel
Copy link
Copy Markdown

vercel Bot commented May 4, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
daily-webapp Ready Ready Preview May 4, 2026 8:50am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
storybook Ignored Ignored May 4, 2026 8:50am

Request Review

@rebelchris rebelchris merged commit 5e6454e into main May 4, 2026
13 checks passed
@rebelchris rebelchris deleted the eng-1358-feedback-feature-request-modal-does-not-close-after-posting branch May 4, 2026 08:57
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.

1 participant