feat(app-check): add Swift AppDelegate support for Expo SDK53+#8521
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
Hi, thanks for working on this! Just wanted to check — is this ready to be used as-is from the branch, or are there still known issues or missing pieces that you're planning to address before merge? Also, is there a target version or timeline for when this might be included in a release? Appreciate the work — looking forward to this being available! |
There was a problem hiding this comment.
This looks great to me, thank you so much for posting the PR
will get this merged and kick out a release, if anyone else sees this and there's a problem with it post-release (perhaps some interaction with the app config plugin? you never know) I will happily take any follow-ons and release them as quickly as I can.
(maintener note to myself: CI is running and should go green on this but we don't normally take Podfile.lock updates from PRs - if there is a hang-up on iOS CI will strip the Podfile.lock commit via rebase, re-push, then merge...)
017ede7 to
14079e8
Compare
|
Had a trivial lint issue, I fixed that and since I was re-pushing anyway, stripped the lockfile update (I do those when updating the test app normally, just my personal preference). Should all go green now but if not will shepherd it through CI with whatever is needed. I'm about to do a release and I want this in there :-). Cheers |
|
Really cool thank you 👍 |
Description
Adding support for Swift AppDelegate for the new version of Expo SDK (53) for Firebase AppCheck.
Checklist
AndroidiOSOther(macOS, web)e2etests added or updated inpackages/\*\*/e2ejesttests added or updated inpackages/\*\*/__tests__Test Plan
Unit Tests:
Test Suites: 39 passed, 39 total
Tests: 1 skipped, 789 passed, 790 total
Snapshots: 31 passed, 31 total
Time: 3.848 s
E2E Tests:
PASS e2e/firebase.test.js (556.894 s)
Jet Tests
✓ runs all tests (527266 ms)
Test Suites: 1 passed, 1 total
Tests: 1 passed, 1 total
Snapshots: 0 total
Time: 556.965 s
Ran all test suites.
🔥