Skip to content

fix: Several small issues reported by Cory#1619

Merged
charlesvien merged 3 commits intomainfrom
04-13-cory-nits
Apr 13, 2026
Merged

fix: Several small issues reported by Cory#1619
charlesvien merged 3 commits intomainfrom
04-13-cory-nits

Conversation

@charlesvien
Copy link
Copy Markdown
Member

@charlesvien charlesvien commented Apr 13, 2026

Problem

Several small issues reported by Cory: command center counts empty windows, logout triggers an error boundary and dev plist still references "Array."

Changes

  1. Exclude cells with no task from command center window count
  2. Move clearAuthScopedQueries to onSuccess so it runs after auth state refreshes
  3. Rename "Array (Development)" to "PostHog Code (Development)" in plist patch script
  4. Hoist taskMap above its first usage in SidebarMenu

How did you test this?

Manually

Copy link
Copy Markdown
Member Author

charlesvien commented Apr 13, 2026

@charlesvien charlesvien marked this pull request as ready for review April 13, 2026 17:20
@charlesvien charlesvien changed the title Exclude empty windows from command center count fix: Several small issues reported by Cory Apr 13, 2026
@charlesvien charlesvien force-pushed the 04-13-replace_task_item_icons_with_clearer_state_indicators branch from ef3e653 to 689ac28 Compare April 13, 2026 18:44
Copy link
Copy Markdown
Member Author

charlesvien commented Apr 13, 2026

Merge activity

  • Apr 13, 7:31 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Apr 13, 7:33 PM UTC: Graphite rebased this pull request as part of a merge.
  • Apr 13, 7:40 PM UTC: @charlesvien merged this pull request with Graphite.

@charlesvien charlesvien changed the base branch from 04-13-replace_task_item_icons_with_clearer_state_indicators to graphite-base/1619 April 13, 2026 19:31
@charlesvien charlesvien changed the base branch from graphite-base/1619 to main April 13, 2026 19:31
@charlesvien charlesvien merged commit 0772f2c into main Apr 13, 2026
16 checks passed
@charlesvien charlesvien deleted the 04-13-cory-nits branch April 13, 2026 19:40
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