Skip to content

fix(inbox): don't auto-redirect after creating PR task from report#2555

Open
rafaeelaudibert wants to merge 1 commit into
mainfrom
posthog-code/pr-task-no-redirect
Open

fix(inbox): don't auto-redirect after creating PR task from report#2555
rafaeelaudibert wants to merge 1 commit into
mainfrom
posthog-code/pr-task-no-redirect

Conversation

@rafaeelaudibert

Copy link
Copy Markdown
Member

Summary

When creating a PR task from the inbox report detail pane, the app no longer auto-redirects to the task detail view. Instead:

  • The task is added to the sidebar (unchanged — via invalidateTasks).
  • The success toast now shows a View task action button that navigates to the task detail page on demand.

Changes

  • useCreatePrReport.ts: dropped the void openTask(output.task) call from the createTask callback, captured the created task, and added a View task action to the success toast.

Test plan

  • pnpm --filter code typecheck
  • pnpm biome check on the changed file
  • Manually create a PR from an inbox report and confirm it stays on the inbox, appears in the sidebar, and the toast's "View task" button opens the task.

Creating a PR task from the inbox report detail pane no longer navigates
away to the task detail page. The task is still added to the sidebar via
invalidateTasks, and the success toast now surfaces a "View task" action
that opens the task on demand.

Generated-By: PostHog Code
Task-Id: 1293530d-3f17-4a5e-9427-f8abae1842c8
@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown

React Doctor found no issues in the changed files. 🎉

Reviewed by React Doctor for commit c1f5421.

@greptile-apps

greptile-apps Bot commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Reviews (1): Last reviewed commit: "fix(inbox): don't auto-redirect after cr..." | Re-trigger Greptile

@rafaeelaudibert rafaeelaudibert requested a review from a team June 9, 2026 22:52
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