-
-
Notifications
You must be signed in to change notification settings - Fork 15
Using this lib results in iOS App Store rejection issue - related to node-sass
dependency
#3
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
Comments
Interesting. I wonder if any of NativeScript's build step hooks could be used to remove this folder? Not sure how to solve either, but will think about it, too. Thanks for reporting the issue! |
Using a webpack build does get away from this issue so I'm going to close this for now. Since using the webpack plugin to create production builds for deployment across the stores should be the standard. |
I'm sure it could benefit others to see what you're doing with webpack, both to resolve this issue and generally improve production builds. If you end-up blogging about it, feel free to share a link here. Probably good fodder for improving the README for this plugin, too. |
Hi guys, I have the same error. How do you use webpack in the build to get round this? |
Now that there is a post-build step in this plugin, I think we could add an additional action to auto-remove this |
That would be awesome |
@dotdreaming @toddanglin this works really well too if you run into webpack struggles: |
Okay. The fix is in and the plugin is version bumped to 0.4.1. The |
Excellent. |
I'm not sure how to solve this just yet, but I ran into this tonight while trying to upload an app to the iOS App Store:
Going into the archive and manually removing the
node-sass/vendor
folder allows the app store submittal to proceed. If a solution comes to mind to fix here, I'll submit a PR, not sure.The text was updated successfully, but these errors were encountered: