Skip to content

refactor(mobile): reuse repository aggregation#3633

Open
richardsolomou wants to merge 3 commits into
posthog-code/share-mobile-cloud-task-corefrom
posthog-code/share-repository-integration-semantics
Open

refactor(mobile): reuse repository aggregation#3633
richardsolomou wants to merge 3 commits into
posthog-code/share-mobile-cloud-task-corefrom
posthog-code/share-repository-integration-semantics

Conversation

@richardsolomou

@richardsolomou richardsolomou commented Jul 21, 2026

Copy link
Copy Markdown
Member

Problem

Mobile independently combined team and user GitHub repository responses, so filtering and partial-failure behavior could drift from desktop/web.

Depends on #3608
Refs #2416

Changes

I switched mobile repository hooks to the existing core aggregation helpers. Picker option labels, cached view state, and pull-to-refresh messaging stay in mobile because they are rendering concerns.

How did you test this?

  • Ran 2,562 core tests and the core typecheck.
  • Ran 413 mobile tests.
  • Ran desktop and web typechecks, Biome, and the host-boundary check.

Automatic notifications

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

Created with PostHog Code

@github-actions

github-actions Bot commented Jul 21, 2026

Copy link
Copy Markdown

React Doctor found no issues in the changed files. 🎉

Reviewed by React Doctor for commit ac5fe9a.

@richardsolomou
richardsolomou force-pushed the posthog-code/share-mobile-cloud-task-core branch from 70e84e9 to dde4532 Compare July 23, 2026 12:59
@richardsolomou
richardsolomou force-pushed the posthog-code/share-repository-integration-semantics branch from 9fa5d73 to d33f9fc Compare July 23, 2026 13:04
@richardsolomou richardsolomou changed the title refactor(mobile): share repository integration semantics refactor(mobile): reuse repository aggregation Jul 23, 2026
@richardsolomou
richardsolomou force-pushed the posthog-code/share-mobile-cloud-task-core branch from dde4532 to 8c45346 Compare July 23, 2026 13:36
@richardsolomou
richardsolomou force-pushed the posthog-code/share-repository-integration-semantics branch from d33f9fc to 34b58d2 Compare July 23, 2026 13:36
@greptile-apps

greptile-apps Bot commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Reviews (1): Last reviewed commit: "Merge posthog-code/share-mobile-cloud-ta..." | Re-trigger Greptile

@richardsolomou
richardsolomou force-pushed the posthog-code/share-repository-integration-semantics branch from 2c40bfe to a7ef739 Compare July 23, 2026 17:28
richardsolomou and others added 3 commits July 23, 2026 21:47
Generated-By: PostHog Code
Task-Id: 40c57a59-b4e1-4760-8e56-ecd03e9c2f0f
Generated-By: PostHog Code
Task-Id: c1bbe3cf-742b-4b24-bf96-d11a18b4cf22
Generated-By: PostHog Code
Task-Id: c1bbe3cf-742b-4b24-bf96-d11a18b4cf22
@richardsolomou
richardsolomou force-pushed the posthog-code/share-repository-integration-semantics branch from a7ef739 to ac5fe9a Compare July 23, 2026 18:55
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