-
-
Notifications
You must be signed in to change notification settings - Fork 50
Fix webpack #39
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
Fix webpack #39
Conversation
CONTRIBUTING.md
Outdated
|
|
||
| * Make commits of logical units. | ||
| * Make sure your commit messages are in the proper format. | ||
| * Make sure your commit messages are in the proper format. We are strongly recommend to use [semantic commit messages](https://seesparkbox.com/foundry/semantic_commit_messages) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove 'are'?
| import { config } from "../../app.config"; | ||
| let http = require("http"); | ||
| let appSettings = require("application-settings"); | ||
| let http = require("tns-core-modules/http"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you can use 'import' instead of 'require'
| import * as Facebook from "nativescript-facebook"; | ||
| import { NavigationService } from "../../services/navigation.service"; | ||
| let appSettings = require("application-settings"); | ||
| let appSettings = require("tns-core-modules/application-settings"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
import
995421b to
2485bf6
Compare
2485bf6 to
c6434fc
Compare
No description provided.