fix(messaging, ios): fixed isHeadless for react-native-navigation

[Beat-YT]

Description

Related issues

fixes #6429

Release Summary

Moved the setter of isHeadless to before the check for rctView that does not exists on react-native-navigation

Checklist

• I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • Yes
• My change supports the following platforms;
  • Android
  • iOS
• My change includes tests;
  • e2e tests added or updated in packages/\*\*/e2e
  • jest tests added or updated in packages/\*\*/__tests__
• I have updated TypeScript types that are affected by my change.
• This is a breaking change;
  • Yes
  • No

Test Plan

I sent a remote notification with content-available to true, and then I also opened the app before it shut itself down to test that it does set itself to false as well.

p.s: Sorry for the double pull request, I read through the CONTRIBUTING.md a bit too fast 😅

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
react-native-firebase	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Sep 3, 2024 9:34am

[Beat-YT]

@mikehardy Can you take a look?

[github-actions]

Hello 👋, this PR has been opened for more than 2 months with no activity on it.

If you think this is a mistake please comment and ping a maintainer to get this merged ASAP! Thanks for contributing!

You have 15 days until this gets closed automatically

[Beat-YT]

Well it’s not my fault if no one looks at it

[Beat-YT]

@mikehardy you said you’d approve such pull request a while back to fix the issue, can you take a look at this please.

[russellwheatley]

@Beat-YT - seems fair enough to me. Need to fix linting failure. There's a script in package.json you need to run: yarn lint:ios:fix. Waiting for CI run to see what else pops up.

[Beat-YT]

@russellwheatley I have fixed linting failure a you requested.

[Beat-YT]

Can you check this out @russellwheatley

[russellwheatley]

As an FYI; I ran app and tested messages in foreground and background which worked as intended. There are no ways to test this e2e so it was a manual check.