Skip to content

refactor(mobile): reuse shared inbox transport#3634

Draft
richardsolomou wants to merge 1 commit into
posthog-code/share-repository-integration-semanticsfrom
posthog-code/share-mobile-inbox-transport
Draft

refactor(mobile): reuse shared inbox transport#3634
richardsolomou wants to merge 1 commit into
posthog-code/share-repository-integration-semanticsfrom
posthog-code/share-mobile-inbox-transport

Conversation

@richardsolomou

Copy link
Copy Markdown
Member

Problem

Mobile maintains a second implementation of the signals inbox HTTP API even though desktop already uses the shared authenticated API client. Endpoint and error behavior can therefore drift between hosts.

Changes

Route mobile inbox queries and mutations through the shared API client used by desktop. Keep only native query wiring, move repository artefact interpretation into core, and remove the duplicated report transport implementation.

How did you test this?

  • Core test suite (2,633 tests)
  • Core typecheck
  • Mobile test suite (340 tests)
  • Biome on changed files
  • Host-boundary check

Automatic notifications

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

Created with PostHog Code

Generated-By: PostHog Code
Task-Id: 40c57a59-b4e1-4760-8e56-ecd03e9c2f0f
@github-actions

Copy link
Copy Markdown

React Doctor found no issues in the changed files. 🎉

Reviewed by React Doctor for commit fcec203.

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