Skip to content

add import loader (since webpack v2.1.0-beta.28) #90

New issue

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

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

Already on GitHub? # to your account

Merged
merged 3 commits into from
Nov 14, 2017
Merged

add import loader (since webpack v2.1.0-beta.28) #90

merged 3 commits into from
Nov 14, 2017

Conversation

JounQin
Copy link
Contributor

@JounQin JounQin commented Nov 10, 2017

@JounQin
Copy link
Contributor Author

JounQin commented Nov 10, 2017

@brandonroberts Please help to review this PR since webpack 3 has removed System.import support.

docs/options.md Outdated

If you prefer to use `dynamic import`, set the `loader` to `import`

**NOTE:** Using `import` only works with Webpack >= 2.1.0-beta.28.
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change to Webpack => 2.1.0 since its already been officially released

docs/options.md Outdated
}
```

If you prefer to use `dynamic import`, set the `loader` to `import`
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove If you prefer

Copy link
Contributor Author

@JounQin JounQin Nov 14, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@brandonroberts I add If you prefer here because it is also mentioned in If you prefer to use System.import, set the loader to system, so should I remove that also?

@brandonroberts
Copy link
Owner

brandonroberts commented Nov 14, 2017

@JounQin will you add a test to verify the import option also? Thanks

@brandonroberts brandonroberts merged commit a9835ab into brandonroberts:master Nov 14, 2017
@brandonroberts
Copy link
Owner

Thanks!

@JounQin JounQin deleted the feat/import-loader branch November 14, 2017 16:57
@brandonroberts
Copy link
Owner

brandonroberts commented Nov 14, 2017

Published as 0.7.0

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants