Skip to content
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

Updated to angular-cli v1.0.0 and angular v4.0.0 #38

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

coder925
Copy link

@coder925 coder925 commented Feb 2, 2017

Updated to angular-cli v1.0.0-beta.28.3 and angular/material v2.0.0-beta.1.

Steps taken:

npm install --save-dev angular-cli@latest
npm install
ng update

During the update the following file actions were taken:

  • skip: README.md
  • skip: src\app\app.component.css
  • skip: src\app\app.component.html
  • skip: src\app\app.component.ts
  • skip: src\app\app.module.ts
  • overwrite: src\environments\environment.prod.ts
  • overwrite: src\environments\environment.ts
  • skip: src\index.html
  • overwrite: src\main.ts
  • overwrite: src\polyfills.ts
  • skip: src\styles.css
  • overwrite: src\test.ts
  • overwrite: src\tsconfig.json
  • merge: angular-cli.json
  • overwrite: .gitignore
  • overwrite: karma.conf.js
  • overwrite: package.json
  • overwrite: protractor.conf.js
  • overwrite: tslint.json
  • added: .editorconfig
  • added: e2e\app.e2e-spec.ts
  • added: e2e\app.po.ts
  • added: e2e\tsconfig.json
  • added: src\app\app.component.spec.ts
  • deleted: src\environments\environment.dev.ts

Created missing folder after update
mkdir src\assets

Updated Material according to documentation:

npm install --save @angular/material
npm install --save hammerjs 
npm install --save-dev @types/hammerjs

Updated firebase-tools:
npm install --save-dev firebase-tools

Renamed classes to align with file naming and allow for easier angular-cli updates in the future.
renamed: MaterialAppModule > AppModule
renamed: Material2AppAppComponent > AppComponent

@bogste
Copy link

bogste commented Feb 3, 2017

Happy to see that you took the care to do it!
I have already done this and was planning a PR.

BTW, how did you get the AoT to work? It looks broken on my side?

@coder925
Copy link
Author

coder925 commented Feb 4, 2017

I am glad you appreciate it. Actually, I did not do anything at all regarding AOT. Since I let angular-cli ng updateoverwrite both main.ts and tsconfig.json some previous AOT related code/config was stripped.

I assumed that the app worked since I get no error messages after running ng serve --aot. Please let me know if I have overlooked something. Thanks!

@coder925 coder925 changed the title Updated to angular-cli v1.0.0-beta.28.3. Updated to angular-cli v1.0.0 and angular v4.0.0 Mar 25, 2017
@coder925
Copy link
Author

Updated to angular-cli v1.0.0, angular v4.0.0 and angular/material v2.0.0-beta.2.

Steps taken:

  1. Created a new app using angular-cli.
    ng new M2App
    npm install --save @angular/material
    npm install --save @angular/animations
    npm install --save hammerjs
    npm install --save-dev firebase-tools
    
  2. Merged the old directory with the new.
  3. Fixed some deprecated syntax in app.component.html and material2-app-theme.scss.

@asiftasleem
Copy link

Please merge this PR or upgrade to latest versions.

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

Successfully merging this pull request may close these issues.

4 participants