[Fabric] iOS - Login - App crashed when try to login.

[kbecciv]

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!

---

Action Performed:

1. Launch App
2. Enter email. password
3. Tap on Sign In

Expected Result:

User able to log in

Actual Result:

App crashed when try to login.

Workaround:

Unknown

Platforms:

Which of our officially supported platforms is this issue occurring on?

• iOS / native

Version Number: 1.2.49.0 Thu, 5 Jan 2023 18:02 PM

Reproducible in staging?: Yes

Reproducible in production?: n/a

If this was caught during regression testing, add the test name, ID and link from TestRail:

Email or phone of affected tester (no customers):

Logs: https://stackoverflow.com/c/expensify/questions/4856

Notes/Photos/Videos: Any additional supporting documentation

Bug5887387_RPReplay_Final1673042074.mp4

Expensify/Expensify Issue URL:

Issue reported by: Applause - Internal Team

Slack conversation:

View all open jobs on GitHub

[OSBotify]

👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

1. Identify the pull request that introduced this issue and revert it.
2. Find someone who can quickly fix the issue.
3. Fix the issue yourself.

[mvtglobally]

NOTE: this issue is specific to #13767 (comment) CC @WoLewicki

[Luke9389]

@mvtglobally, do you mean this is happening on the branch from that PR?

[mvtglobally]

@Luke9389, correct! It is NOT a staging app issue

[Luke9389]

Given that the linked PR is a draft, should we delete this issue?

[mvtglobally]

@Luke9389 Dont delete it pls. There is a special request for this. Tagged you in Slack.

[Luke9389]

Oh ok cool, thanks @mvtglobally!

[mvtglobally]

@roryabraham @AndrewGable Let us know if any special tagging required for this issue.
This is coming from "Enable Fabric and TurboModules on iOS & Android changes "

[Luke9389]

Just spoke with @mvtglobally 1:1 on slack, and the context for this issue is here: https://expensify.slack.com/archives/C9YU7BX5M/p1672784184899339

Looks like we're performing regression tests for a build on the branch of the linked PR.

For that reason I don't think this is a deploy blocker.

[AndrewGable]

@WoLewicki - Please comment so I can assign you, this is the crash for Fabric.

[WoLewicki]

I'll be working on the issue.

[j-piasecki]

I will be working alongside @WoLewicki.

[melvin-bot]

Whoops! This issue is 2 days overdue. Let's get this updated quick!

[j-piasecki]

This issue should be fixed by WoLewicki@6e6f84f which updates the commit urbanairship-react-native is installed from to j-piasecki/urbanairship-react-native-module@2d69565.

The issue was coming from the fact that enableUserPushNotifications method was not present on the iOS module of UrbanAirship.

[melvin-bot]

@j-piasecki, @WoLewicki Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

[MelvinBot]

This issue has not been updated in over 14 days. @j-piasecki, @WoLewicki eroding to Weekly issue.

[roryabraham]

Looks like all that's left here is to circle back and make sure that this bug is fixed on a new fabric build

[MelvinBot]

This issue has not been updated in over 15 days. @j-piasecki, @WoLewicki eroding to Monthly issue.

P.S. Is everyone reading this sure this is really a near-term priority? Be brave: if you disagree, go ahead and close it out. If someone disagrees, they'll reopen it, and if they don't: one less thing to do!

[melvin-bot]

@j-piasecki, @WoLewicki, this Monthly task hasn't been acted upon in 6 weeks; closing.

If you disagree, feel encouraged to reopen it -- but pick your least important issue to close instead.

[melvin-bot]

@j-piasecki, @WoLewicki, this Monthly task hasn't been acted upon in 6 weeks; closing.

If you disagree, feel encouraged to reopen it -- but pick your least important issue to close instead.