objecttothis

Results 411 comments of objecttothis

> > > I see what you mean, you might want to have something minimal working first in that case so you can start using it already. This PR as...

@jekkos can we sort out whether EventSauce is what we want to use for this instead of CI Events?

@daN4cat @jekkos I wouldn't mind tackling this. I was just reading the documentation (https://codeigniter.com/user_guide/installation/upgrade_4xx.html) and while it's more work than other incremental changes, it appears to be primarily syntax changes....

Implications would mean that our minimum PHP version would be 7.3.x. Currently OSPOS has a 7.2.x minimum for PHP. According to CI migrations after 3.x to 4.x are simpler.

@opensourcepos/org an optional CI4 change in the Views is to change the `echo` syntax from `` to `` So for example, `` would become `` IMO this is cleaner, therefore...

OK, sounds good. As is, it is A LOT of refactoring. I think the more help I get with regression testing and figuring out bugs, the faster this will go.

> > > Let me know if there is anything I can do to help out Once I get it to a place where it builds I'll need someone to...

> Is this all going into 1 request or application? I suppose I will need to build it with composer This PR will only include what is necessary to get...

> Also are you upgrading Bootstrap If compatible I will not upgrade bootstrap here but leave that for someone to do in a later pr.

> @jekkos unless released I would kick out 3.4.0 or whatever is the version and make CI4 for a new release. > It's either OSPOS 4.0.0 or 3.5.0. > >...