Skip to content

feat(shell): restore the last app screen after restart - #3965

Merged
trunk-io[bot] merged 8 commits into
mainfrom
posthog-code/restore-startup-location
Jul 31, 2026
Merged

feat(shell): restore the last app screen after restart#3965
trunk-io[bot] merged 8 commits into
mainfrom
posthog-code/restore-startup-location

Conversation

@puemos

@puemos puemos commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Problem

Restarting the app loses the user’s place, while first-time or invalid destinations should begin with a task in their personal space.

Changes

  • Restore the last stable route per project after startup.
  • Validate saved task and space routes, falling back to a new task in me when unavailable.

Why: Keep restarts continuous for returning users and give everyone else an immediately useful starting point.

How did you test this?

  • pnpm --filter @posthog/ui exec vitest run src/router/startupLocation.test.ts
  • pnpm --filter @posthog/ui typecheck
  • pnpm exec biome check ...
  • node scripts/check-host-boundaries.mjs

Automatic notifications

  • Publish to changelog?
  • Alert Sales and Marketing teams?

Created with PostHog Code

Generated-By: PostHog Code
Task-Id: c6ae4672-c246-4078-857d-b00e81aaaaaa
@trunk-io

trunk-io Bot commented Jul 29, 2026

Copy link
Copy Markdown

😎 Merged successfully - details.

@github-actions

github-actions Bot commented Jul 29, 2026

Copy link
Copy Markdown

React Doctor found no issues in the changed files. 🎉

Reviewed by React Doctor for commit f10c73a.

Generated-By: PostHog Code
Task-Id: c6ae4672-c246-4078-857d-b00e81aaaaaa
@puemos puemos changed the title Restore the last app screen after restart feat(shell): restore the last app screen after restart Jul 29, 2026
puemos added 4 commits July 29, 2026 19:28
Generated-By: PostHog Code
Task-Id: c6ae4672-c246-4078-857d-b00e81aaaaaa
Generated-By: PostHog Code
Task-Id: c6ae4672-c246-4078-857d-b00e81aaaaaa
Generated-By: PostHog Code
Task-Id: c6ae4672-c246-4078-857d-b00e81aaaaaa
Generated-By: PostHog Code
Task-Id: c6ae4672-c246-4078-857d-b00e81aaaaaa
@posthog

posthog Bot commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

🦔 ReviewHog reviewed this pull request

Found 0 must fix, 1 should fix, 2 consider.

Published 3 findings (view the review).

@posthog

posthog Bot commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

ReviewHog Alpha 🦔 If you find any issues helpful - please reply "valid", "invalid", etc., for evaluation purposes 🙏

@posthog posthog Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ReviewHog Report

Changes

Issues: 3 issues

Files (4)
  • packages/ui/src/features/canvas/ensurePersonalChannel.ts
  • packages/ui/src/shell/App.tsx
  • packages/ui/src/shell/rendererStorage.ts
  • packages/ui/src/shell/startupLocation.ts

Comment thread packages/ui/src/features/canvas/ensurePersonalChannel.ts Outdated
Comment thread packages/ui/src/shell/startupLocation.ts
Comment thread packages/ui/src/shell/startupLocation.ts
Generated-By: PostHog Code
Task-Id: c6ae4672-c246-4078-857d-b00e81aaaaaa
@posthog

posthog Bot commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Visual changes approved by @puemos — baseline updated in f10c73a.

View this run in PostHog

6 changed, 4 removed.

6 updated, 4 removed
Run: e0e79852-4378-4765-b314-bf9b47cebef4

Co-authored-by: puemos <13174025+puemos@users.noreply.github.com>

@tatoalo tatoalo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

noice

@puemos
puemos marked this pull request as ready for review July 31, 2026 09:24
@puemos

puemos commented Jul 31, 2026

Copy link
Copy Markdown
Contributor Author

/trunk merge

@trunk-io
trunk-io Bot merged commit b939358 into main Jul 31, 2026
38 checks passed
@trunk-io
trunk-io Bot deleted the posthog-code/restore-startup-location branch July 31, 2026 09:39
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