Skip to main content ↑ ↓
EARLY Beta version of 2.0 Started by Administrator · · Read 310919 times 0 Members and 1 Guest are viewing this topic. previous topic - next topic

Re: EARLY Beta version of 2.0

Reply #135 –

Quote from: "Burke Knight" – Just going to wait until you post a beta installer before I test it....lol
That is a good idea LOL ... and soon-ish ;)

I did another push of updates this PM, really just some housekeeping tasks that needed to get done before the B1 package. 

The only "major" change was that I broke up the PM controller (note its still a  :poop: show) but getting better.  I pulled out labels, filters, reports, and searching to their own controllers.  Yes all of those functions were contained in the single gigantic pm controller.  At least now the various functions of PM are in their own controllers so updates and enhancements should be easier.  It cut the size of the primary controller in 1/2 which is good but its still way to long:man_shrugging:

Re: EARLY Beta version of 2.0

Reply #136 –

Another small bump ....

Update the emoji image packages to v16/v17 (unicode) they were at v14 ... so that added a few new ones.  Also chose to use GitHub naming for the :name: can't remember what was used the first time, but there are really no defacto standards ...

Fixed a strange dispatch path that allowed the dispatcher to maybe bypass the action controller which was not noticeable but could have some less than ideal side effects.

Fixed a couple of minor issues from the previous PM refactor (actually one issue was pre ElkArte) due to some wrong boolean math.  Ah well ... generally harmless.

OK back to getting the installer finished!
 

Re: EARLY Beta version of 2.0

Reply #137 –

Wonder if one fix broke the marking of notifications read function, as they wont clear now.

Re: EARLY Beta version of 2.0

Reply #138 –

Quote from: Burke Knight – Wonder if one fix broke the marking of notifications read function, as they wont clear now.
I'd say yes ... clears when you select the notification from the list but not when you click mark read .... easy fix.

I'm working on one of the last beta changes, reorganize the admin menu.  When I push that to the site I'll fix the notification item.