Closed shlomiassaf closed 8 years ago
I'll check it out on Monday. Thanks for the hard work.
On Oct 1, 2016 8:05 PM, "Shlomi Assaf" notifications@github.com wrote:
@isaacplmann https://github.com/isaacplmann This is the PR with you commit inside (squashed)
Please review.
Thanks!
You can view, comment on, or merge this pull request online at:
https://github.com/shlomiassaf/angular2-modal/pull/211 Commit Summary
- misc: refactor code base in preparation for AoT support
- refactor(build): refactor to support AoT
File Changes
- M .gitignore https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-0 (3)
- M README.md https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-1 (2)
- M config/webpack.common.js https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-2 (11)
- M gulp/bundle.js https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-3 (2)
- M gulp/config.js https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-4 (18)
- M gulp/relative-imports.js https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-5 (40)
- M gulp/scripts.js https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-6 (104)
- M gulpfile.js https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-7 (82)
- M package.json https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-8 (46)
- M src/components/angular2-modal/components/css-backdrop.ts https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-9 (2)
- M src/components/angular2-modal/framework/createComponent.ts https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-10 (2)
- M src/components/angular2-modal/overlay/overlay.component.ts https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-11 (4)
- M src/components/angular2-modal/plugins/bootstrap/bootstrap. module.ts https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-12 (14)
- M src/components/angular2-modal/plugins/bootstrap/index.ts https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-13 (2)
- M src/components/angular2-modal/plugins/bootstrap/message- modal.component.ts https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-14 (2)
- M src/components/angular2-modal/plugins/bootstrap/presets/ message-modal-preset.ts https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-15 (1)
- M src/components/angular2-modal/plugins/js-native/index.ts https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-16 (2)
- M src/components/angular2-modal/plugins/js-native/js-native. module.ts https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-17 (14)
- M src/components/angular2-modal/plugins/vex/dialog-form-modal.ts https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-18 (4)
- M src/components/angular2-modal/plugins/vex/index.ts https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-19 (2)
- M src/components/angular2-modal/plugins/vex/presets/dialog- preset.ts https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-20 (1)
- M src/components/angular2-modal/plugins/vex/vex.module.ts https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-21 (14)
- M src/components/angular2-modal/providers/dom-modal-renderer.ts https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-22 (2)
- M src/demo/app/bootstrap-demo/bootstrap-demo-page/bootstrap- demo-page.ts https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-23 (2)
- M src/demo/app/bootstrap-demo/bootstrap-demo-page/custom- modal-sample.ts https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-24 (1)
- M src/demo/app/bootstrap-demo/bootstrap-demo.ts https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-25 (4)
- M src/demo/app/bootstrap-demo/modal-customisation-wizard/ modal-customisation-wizard.ts https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-26 (4)
- M src/demo/app/home/home.ts https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-27 (2)
- M src/demo/app/home/in-app-plugin/index.ts https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-28 (2)
- M src/demo/app/home/in-app-plugin/modal-backdrop.ts https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-29 (4)
- M src/demo/app/js-native-demo/js-native-demo.ts https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-30 (4)
- M src/demo/app/vex-demo/login-dialog.ts https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-31 (2)
- M src/demo/app/vex-demo/vex-demo.ts https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-32 (9)
- M src/polyfills.ts https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-33 (2)
- M src/vendor.ts https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-34 (1)
- M tsconfig.json https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-35 (17)
- R tsconfig.release.json https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-36 (34)
- D typings/browser.d.ts https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-37 (5)
- D typings/browser/ambient/es6-shim/index.d.ts https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-38 (670)
- D typings/browser/ambient/node/index.d.ts https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-39 (2082)
- D typings/browser/ambient/webpack/index.d.ts https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-40 (265)
- D typings/browser/definitions/es6-promise/index.d.ts https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-41 (84)
- D typings/browser/definitions/zone.js/index.d.ts https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-42 (445)
- D typings/main.d.ts https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-43 (5)
- D typings/main/ambient/es6-shim/index.d.ts https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-44 (670)
- D typings/main/ambient/node/index.d.ts https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-45 (2082)
- D typings/main/ambient/webpack/index.d.ts https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-46 (265)
- D typings/main/definitions/es6-promise/index.d.ts https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-47 (84)
- D typings/main/definitions/zone.js/index.d.ts https://github.com/shlomiassaf/angular2-modal/pull/211/files#diff-48 (445)
Patch Links:
- https://github.com/shlomiassaf/angular2-modal/pull/211.patch
- https://github.com/shlomiassaf/angular2-modal/pull/211.diff
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/shlomiassaf/angular2-modal/pull/211, or mute the thread https://github.com/notifications/unsubscribe-auth/AA0lQIScK7lZIpL12rtrORquoU2WHKSSks5qvvVWgaJpZM4KL5Wv .
Looks good to me. 2.0.0 compiles in the AoT compiler in my app. I'm looking forward to the day when the tooling around ng2 settles down a bit and we don't have to rewrite the build process every 2 months.
I spoke too soon, my app doesn't compile with angular2-modal 2.0.0. I'll file a proper issue in a bit.
Having a tough morning. After clearing the cache, everything is fine. No issue from me.
@isaacplmann This is the PR with you commit inside (squashed)
Please review.
Thanks!