Re: [DISCUSS] Add ngrx to handle state management in Angular

2018-11-26 Thread James Sirota
I found a helpful article here: https://brianflove.com/2018/01/08/ngrx-the-basics/ A lot of this goes over my head, but in a nutshell, it's a tree-based state management object for JS. Its main drawback seems to be added complexity, but if the guys who are more familiar with UI say we would ben

Re: [DISCUSS] Add ngrx to handle state management in Angular

2018-11-26 Thread Michael Miklavcic
Shane, thanks for sharing this. Can you perhaps describe a sample use case in the UI currently and explain for us how it currently works (or doesn't, ha) versus how it would be modified and improved with using NgRx? Thanks, Mike On Fri, Nov 23, 2018 at 7:44 AM Shane Ardell wrote: > What I'm ref

Re: [DISCUSS] Add ngrx to handle state management in Angular

2018-11-26 Thread Michael Miklavcic
Just an fyi for newer members of the dev community - we are very open to and appreciative of all community involvement and feedback - for non-committers, as a general rule please make a small note next to any votes (-1's, +1's, etc.) indicating "(non-binding)". On Mon, Nov 26, 2018 at 7:04 AM Tamá

Re: Unzipping Cypress

2018-11-26 Thread Shane Ardell
It seems we can pretty easily configure the .travis.yml config file to cache our npm modules: https://docs.cypress.io/guides/guides/continuous-integration.html#Caching-the-Cypress-binary It also looks like we are already trying to cache our npm modules in the Travis config, but, obviously, it's no

Re: Unzipping Cypress

2018-11-26 Thread Michael Miklavcic
Shane, Tibor - Can you guys chime in on this? On Mon, Nov 26, 2018 at 9:13 AM Otto Fowler wrote: > Isn’t there a way we can cache it? > > > On November 26, 2018 at 10:59:20, Nick Allen (n...@nickallen.org) wrote: > > Yes, I have noticed that too. If not a way to reduce the time, we should > not

Re: Unzipping Cypress

2018-11-26 Thread Otto Fowler
Isn’t there a way we can cache it? On November 26, 2018 at 10:59:20, Nick Allen (n...@nickallen.org) wrote: Yes, I have noticed that too. If not a way to reduce the time, we should not be logging the unzipping process percentile-by-percentile in the Travis CI builds. On Sat, Nov 24, 2018 at 9:4

Re: Unzipping Cypress

2018-11-26 Thread Nick Allen
Yes, I have noticed that too. If not a way to reduce the time, we should not be logging the unzipping process percentile-by-percentile in the Travis CI builds. On Sat, Nov 24, 2018 at 9:49 AM Otto Fowler wrote: > Anyone else seeing a lot of time taken downloading and unzipping Cypress on > buil

Re: [DISCUSS] Add ngrx to handle state management in Angular

2018-11-26 Thread Tamás Fodor
That is a great idea! +1 On Mon, Nov 26, 2018 at 10:19 AM Tibor Meller wrote: > Shane, Thanks for gathering the information and raising this. > I also feel that our UIs reached a level of complexity what makes this to a > reasonable next step. > This complexity on the client side will grow in t

Re: [DISCUSS] Add ngrx to handle state management in Angular

2018-11-26 Thread Tibor Meller
Shane, Thanks for gathering the information and raising this. I also feel that our UIs reached a level of complexity what makes this to a reasonable next step. This complexity on the client side will grow in the future and I believe it is better to prepare instead of trying to make huge refactoring