lathonez / clicker

Ionic 2 + @angular/cli Seed Project : Angular2 + Typescript + Karma + Protractor + Travis
http://lathonez.com/2018/ionic-2-unit-testing/
MIT License
430 stars 137 forks source link

Simple Internationalization #271

Closed stonelasley closed 6 years ago

stonelasley commented 6 years ago

I'll have to look into circleci but Saturday is calling. AppVeyor is stalled out asking for ionic/cli-plugin-ionic-angular so it probably needs to be updated beyond 3.4.0. @lathonez if you see this while it's broken It would be great if you could try rebuilding on circleci without cache.

lathonez commented 6 years ago

@stonelasley - looks good mate but it'll be a while until I have time to take a look.

hfwittmann commented 6 years ago

I am confused what I should do now, I suppose I'll wait in the wings until this one is in...

stonelasley commented 6 years ago

@hfwittmann sorry if I stepped on your toes with this PR.

stonelasley commented 6 years ago

Ok, could use a little help on this one. CircleCI I'm not going to pursue until we can try a build without cache, although we may see it fail on the e2e tests just like app veyor. I'm having a hell of a time reproducing these results locally so I apologize for the PR spam. The problem is the check is happening before the title has been translated. browser.waitForAngular() seems to work on my machine but obviously not on appveyor. I think the simplest solution is to introduce another delay but there has to be a more elegant solution?

hfwittmann commented 6 years ago

@stonelasley no problem

stonelasley commented 6 years ago

Killing PR again to troubleshoot e2e test

lathonez commented 6 years ago

@stonelasley

- run:
          name: install deps
          command: npm install

The dependencies are baked into the Dockerfile, I'll update it when I merge this in. The abvoe should get circle passing I think.

stonelasley commented 6 years ago

@lathonez excellent thank you!

stonelasley commented 6 years ago

@lathonez, I should be able to sort these out this week. Thanks for the comments.

lathonez commented 6 years ago

Will rebase this and get it merged this week

lathonez commented 6 years ago

280