Skip to content

Comments

Feat/3080 apns token registration#3101

Open
adrastaea wants to merge 23 commits intodevelopfrom
feat/3080-apns-token-registration
Open

Feat/3080 apns token registration#3101
adrastaea wants to merge 23 commits intodevelopfrom
feat/3080-apns-token-registration

Conversation

@adrastaea
Copy link
Collaborator

@adrastaea adrastaea commented Jan 30, 2026

Passes APNS device token through state-manager socket with backend, caches token until ready to register, and posts to the register endpoint.

Pull Request Checklist

  • I have linked this PR to a related GitHub issue.
  • I have added a description of the change (and Github issue number, if any) to the root CHANGELOG.md.

(Optional) Mobile checklist

Please ensure you completed the following checks if you did any changes to the mobile package:

  • I have run e2e tests for mobile
  • I have updated base screenshots for visual regression tests

@adrastaea adrastaea linked an issue Jan 30, 2026 that may be closed by this pull request
5 tasks
@adrastaea
Copy link
Collaborator Author

Based on #3100, merge that first.

@mcginty
Copy link
Collaborator

mcginty commented Feb 2, 2026

Just noting that if we're going to use Firebase as our APNS intermediary, there are a few extra hookup steps a la https://github.com/TryQuiet/ios-notification-experiments/blob/main/ios-decrypt-test/ios_decrypt_testApp.swift#L24-L110

@adrastaea adrastaea requested a review from islathehut February 9, 2026 23:51
Comment on lines 72 to 73
// Fall back to HTTP
return this._registerViaHttp(deviceToken)
Copy link
Collaborator

Choose a reason for hiding this comment

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

no need for this now i believe!

Comment on lines +12 to +15
export interface QPSRegisterPayload {
deviceToken: string
bundleId: string
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

same as QPSRegisterRequest above, are both needed?

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.

Register APNS Device Token with QPS

2 participants