Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

1.3.2 [Bug]: 'onesignal/onesignal.h' file not found react native #182

Closed
1 task done
MortadhaFadhlaoui opened this issue Apr 27, 2023 · 8 comments · Fixed by #195
Closed
1 task done

1.3.2 [Bug]: 'onesignal/onesignal.h' file not found react native #182

MortadhaFadhlaoui opened this issue Apr 27, 2023 · 8 comments · Fixed by #195

Comments

@MortadhaFadhlaoui
Copy link

What happened?

The latest version of the SDK 1.3.2 cause ios build to fail

Steps to reproduce?

[email protected]
npx expo run:ios

What did you expect to happen?

finish ios build successfully

OneSignal Expo SDK version

1.3.2

Platform

iOS

Relevant log output

❌  (ios/OneSignalNotificationServiceExtension/NotificationService.m:1:9)

> 1 | #import <OneSignal/OneSignal.h>
    |         ^ 'OneSignal/OneSignal.h' file not found
  2 | 
  3 | #import "NotificationService.h"

Code of Conduct

  • I agree to follow this project's Code of Conduct
@MortadhaFadhlaoui MortadhaFadhlaoui changed the title [email protected] [Bug]: 'onesignal/onesignal.h' file not found react native 1.3.2 [Bug]: 'onesignal/onesignal.h' file not found react native Apr 27, 2023
@emawby
Copy link
Contributor

emawby commented Apr 27, 2023

@MortadhaFadhlaoui Was this working on a previous version and is failing now that you have upgraded? If so which version were you on previously? Has anything else changed in your build process (Mac version, Xcode version, Cocoapods version, etc.)?

@MortadhaFadhlaoui
Copy link
Author

MortadhaFadhlaoui commented Apr 28, 2023

@emawby nothing has changed so far, I have the same build environment in EAS

Installed software:
- Xcode 14.1 (14B47b)
- Node.js 16.18.1
- Yarn 1.22.17
- pnpm 7.15.0
- npm 8.1.2
- fastlane 2.210.1
- CocoaPods 1.11.3
- Ruby 2.7
- Expo CLI 6.0.0

I only downgrade to 1.3.1 and the build is working again.
Even in my local machine, nothing changed.

@emawby
Copy link
Contributor

emawby commented Apr 28, 2023

It looks like the 1.3.2 release only bumped dependency versions and then made this change. That PR is probably causing the issue 👀

@TNAJanssen
Copy link

Same issue here, even after downgrading..

@maxtor3569
Copy link

Same issue here. 1.3.2 / 1.3.1 is failing / 1.3.0 too.

@maxtor3569
Copy link

To fix you have to replace the code in NotificationService like in the doc..
https://documentation.onesignal.com/docs/react-native-sdk-setup

@rgomezp
Copy link
Contributor

rgomezp commented Sep 11, 2023

Looks like the solution is to replace the import in the NotificationService.m file:

OneSignal/react-native-onesignal#1440 (comment)

This will be fixed in https://github.com/OneSignal/onesignal-expo-plugin/pull/195/files

@Diaa-Ghonim
Copy link

same issue here any idea??

OneSignal/react-native-onesignal#1440 (comment)

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 a pull request may close this issue.

6 participants