-
-
Notifications
You must be signed in to change notification settings - Fork 323
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
Getting "No platform selected" from sentry-wizard #1386
Comments
Thanks for reporting, we are taking a look |
I am seeing the exact same issue. Project previously had v2.0.0 setup in November and everything worked well. After updating via yarn to 2.3.0 I started getting compile issues in Android. Wizard --uninstall would fail with ENAMETOOLONG when trying to revert the iOS project, so I just reverted my initial sentry install commit, cleaned build folders, ran yarn, etc. Then when trying to re-add sentry using the current version and wizard I get the same error as the OP. Of note, I get asked to select the platform 5 times during the wizard. IOS and Android are selected each time. Using -p ios android throws the same error, but doesn't ever ask for the platforms. How do you use Sentry? Which SDK and version?
|
+1 |
Facing the same issue, @sentry/react-native: 2.4.0 This would likely be a reason source maps are not being uploaded right? |
The same issue |
Same here. |
Same here |
same here |
Had the same issue.
And "An organization slug is required" error on building project. Then I just copied sentry.properties from android to ios directory and everything works fine. |
Indeed as @victoriaSh said, I too am stuck on this new TypeError. |
I decided not to be lazy and used the manual configuration. It is now working for me. Hopefully other people will get to enjoy the wizard. |
I'm on React Native 0.59
I can't even "pod install" now. Throws this:
Error
|
Same issue here. |
I can confirm that the PR 105 fixes the issue. Thanks! |
@hanselsen how can i get the version with that PR merged ? |
@KinanKazmi You can just update to the latest version. |
It says last update was 6 days ago. I just tried it last hour.
…On Mon, May 10, 2021, 11:16 AM Jenn Mueng ***@***.***> wrote:
@KinanKazmi <https://github.com/KinanKazmi> You can just update to the
latest version.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1386 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AE2HKU3YV3EVDRCGTUM7FL3TM523FANCNFSM4Y5TFVDQ>
.
|
@KinanKazmi Did you actually update? This issue does not exist anymore. |
@jennmueng i just installed it an hour ago and got that error. |
If it still occurs please provide a reproduction and the version in your lockfile/node_modules |
@KinanKazmi try |
@hanselsen some of my packages dont exist on yarn so that doesn't work at all. |
@jennmueng can you confirm if i need to provide more info for this ? |
@jennmueng just tried it for a brand new project. Still fails.
## ### #### ### ###### ######## ####### ########## ####### ######## ###### ### #### ### ##
✔ Downloading template kinan@Kinans-MacBook-Pro testForSentry % npm install --save @sentry/react-native added 107 packages, changed 3 packages, and audited 1072 packages in 9s 72 packages are looking for funding 4 low severity vulnerabilities Some issues need review, and may require choosing Run Please open ? Please select your project in Sentry: Avaza / avaza-expenses-apps 🎉 Successfully set up Sentry for your project 🎉 |
@KinanKazmi |
@hanselsen same, even for a brand new project |
@KinanKazmi if it says that it patched the files successfully, you should be good. I will check why the error still throws if I can reproduce your issue. |
@jennmueng it actually broke my xcode project. The xCode cant even open the project. Luckily i had Git backup so i restored it. Then pod install caused more issues. Updated cocapods and still no difference. |
@KinanKazmi Can you provide a reproduction repo? Looks like some edge case as it’s solved for others. |
@jennmueng its happening with my new project too.
Result is the same screenshot i posted above. |
@KinanKazmi Try |
@jennmueng sorry i forgot to respond here, yes that worked. Thanks a bunch! |
Environment
How do you use Sentry?
Sentry SaaS
Which SDK and version?
@sentry/react-native: 2.2.2
react-native: 0.63.3
Steps to Reproduce
npx @sentry/wizard -i reactNative -p android
`✅ Patched build.gradle file.
Error: No platform selected
at ReactNative.MobileProject.getPlatforms (/Applications/MAMP/htdocs/shkolo/shkolo-mobileapp/node_modules/@sentry/wizard/dist/lib/Steps/Integrations/MobileProject.js:65:19)
at ReactNative._patchJs (/Applications/MAMP/htdocs/shkolo/shkolo-mobileapp/node_modules/@sentry/wizard/dist/lib/Steps/Integrations/ReactNative.js:224:14)
at /Applications/MAMP/htdocs/shkolo/shkolo-mobileapp/node_modules/@sentry/wizard/dist/lib/Helper/File.js:28:45
at processTicksAndRejections (node:internal/process/task_queues:94:5)
🎉 Successfully set up Sentry for your project 🎉
3. But android/sentry.properties is not generated and after that when I try to build the android app I get:
Expected Result
The Sentry setup should be successful and the app should be build successfully
Actual Result
I get both errors described in the steps.
The text was updated successfully, but these errors were encountered: