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

Updating Angular to version 1.6 #11857

Merged
merged 7 commits into from Sep 11, 2017
Merged

Updating Angular to version 1.6 #11857

merged 7 commits into from Sep 11, 2017

Conversation

diosmosis
Copy link
Member

This PR can be reviewed commit by commit.

Not sure if this can be merged, I wanted to use .component() which isn't available in 1.4.

Changes:

  • Updated angular & related packages to version 1.6.
  • Updated ngDialog to latest version.

Notes:

Angular 1.6 should be compatible w/ IE9+, but I haven't tested manually yet. If there's anything that should be tested manually on certain browsers, let me know.

@diosmosis diosmosis added the Needs Review PRs that need a code review label Jul 11, 2017
@diosmosis diosmosis requested review from tsteur and mattab July 11, 2017 03:02
@mattab mattab added this to the 3.1.0 milestone Jul 11, 2017
@mattab
Copy link
Member

mattab commented Jul 11, 2017

UI tests are passing which is promising. @sgiehl maybe you have some feedback as well. we could merge this as soon as 3.0.5 is released and then cut a beta to check whether all plugins builds will pass?

@sgiehl
Copy link
Member

sgiehl commented Jul 11, 2017

@mattab Will have a closer look the next days, but let's merge after 3.0.5 as you suggested.

@diosmosis
Copy link
Member Author

Did a quick check on a VM w/ IE9, seems broken on 3.x-dev, not sure why. (Also broken on demo.piwik.org).

Some pages don't load (like the dashboard), some look odd, like the visitors log. Known issues?

@diosmosis
Copy link
Member Author

diosmosis commented Jul 17, 2017

Noticed the screenshots aren't stored in LFS, need to fix that before this can be merged.

Nevermind, github LFS notification isn't everywhere.

@tsteur
Copy link
Member

tsteur commented Jul 17, 2017

FYI: We dropped IE9 support with Piwik 3

@diosmosis
Copy link
Member Author

That's good, I'll check on ie10 :)

@diosmosis
Copy link
Member Author

Looks like it's working on ie10. Did a cursory check of dialogs via row actions + a couple pages + the top controls.

@mattab
Copy link
Member

mattab commented Jul 24, 2017

Good to hear @diosmosis - we'll merge this in the coming days and cut a beta then

@mattab
Copy link
Member

mattab commented Sep 11, 2017

We just released 3.1.0 so now merging this PR so we have time to test in the beta cycle

@mattab mattab merged commit 1092e4d into matomo-org:3.x-dev Sep 11, 2017
@diosmosis diosmosis deleted the angular-1.6 branch September 11, 2017 17:42
@mattab mattab modified the milestones: 3.2.0, 3.1.1 Sep 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Review PRs that need a code review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants