Skip to content

[DO NOT MERGE] fix(mobile): give the widget and notification-service targets a team id - #131

Closed
GSTJ wants to merge 2 commits into
mainfrom
fix/ios-extension-target-signing
Closed

GSTJ wants to merge 2 commits into
mainfrom
fix/ios-extension-target-signing

Conversation

@GSTJ

@GSTJ GSTJ commented Aug 2, 2026

Copy link
Copy Markdown
Owner

Summary

Reopens #109 with the same diff, merged without review and reverted in #124. Only makes sense again together with the widget and notification PRs.

Gabriel Taveira added 2 commits July 27, 2026 20:20
@bacons/apple-targets reads ios.appleTeamId to stamp DEVELOPMENT_TEAM onto
the targets it generates. The field was never set, so prebuild took the "no
team anywhere" branch: it strips DEVELOPMENT_TEAM from every target and
writes the literal `DevelopmentTeam = undefined;` into TargetAttributes.
It also warns "iOS builds may fail until this is corrected" on every CI
prebuild. Sourced from EXPO_APPLE_TEAM_ID so the id stays out of the repo.

Also corrects release-mobile.yml's CREDENTIALS block, which predicted the
wrong failure. EAS sets iOS credentials up per target, and in non-interactive
mode SetUpDistributionCertificate can only reuse a certificate already
assigned to that target: no branch adopts an unassigned account certificate
and none creates one. An ASC API key does not help, since the rc1 run was
already using one when it failed at the certificate lookup, and it cannot
link App Group identifiers at all. The build step now names the target and
the command to run instead of leaving eas-cli's bare "run this command again
in interactive mode".

Refs #104

Claude-Session: https://claude.ai/code/session_01QsVEsfkynXpJWK4t33exG9
…ne is absent

pipefail plus the job shell's -e meant a no-match grep would kill the step
before it printed the annotation it exists to print.

Refs #104

Claude-Session: https://claude.ai/code/session_01QsVEsfkynXpJWK4t33exG9
@GSTJ

GSTJ commented Aug 3, 2026

Copy link
Copy Markdown
Owner Author

The extension targets are staying out, so this signing patch has nothing left to fix.

@GSTJ GSTJ closed this Aug 3, 2026
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