search

VidYen / VidYen-WordPress-Plugins

VidYen Plugins for WordPress
GNU General Public License v2.0
9 stars 2 forks source link

Point Log load records #116

Open mcdeimon opened 5 years ago

mcdeimon commented 5 years ago

I think it would be nice to not load all the records in the Point Log from the administration area on a single page. It would be nice to show 590 results per page or something like that. Now every time you access the Point Log you have to load all the records and that consumes resources unnecessarily.

Thanks!

VidYen commented 5 years ago

Yeah. I had thought of that but haven’t implemented it. You can use public log somewhere and not make it public unless you want to make it public.

On Sun, Oct 14, 2018 at 5:01 PM mcdeimon notifications@github.com wrote:

I think it would be nice to not load all the records in the Point Log from the administration area on a single page. It would be nice to show 590 results per page or something like that. Now every time you access the Point Log you have to load all the records and that consumes resources unnecessarily.

Thanks!

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/VidYen/VYPS-Plugin/issues/116, or mute the thread https://github.com/notifications/unsubscribe-auth/AmNhDbYq2-coQ3-UFpoOa2S5RTk4vJTzks5uk6YpgaJpZM4XbVGY .

VidYen commented 5 years ago

And yes I do plan on adding it to the admin page as my own test servers have quite a log now. :)

On Sun, Oct 14, 2018 at 5:03 PM James Felty Douglas james.felty@vidyen.com wrote:

Yeah. I had thought of that but haven’t implemented it. You can use public log somewhere and not make it public unless you want to make it public.

On Sun, Oct 14, 2018 at 5:01 PM mcdeimon notifications@github.com wrote:

I think it would be nice to not load all the records in the Point Log from the administration area on a single page. It would be nice to show 590 results per page or something like that. Now every time you access the Point Log you have to load all the records and that consumes resources unnecessarily.

Thanks!

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/VidYen/VYPS-Plugin/issues/116, or mute the thread https://github.com/notifications/unsubscribe-auth/AmNhDbYq2-coQ3-UFpoOa2S5RTk4vJTzks5uk6YpgaJpZM4XbVGY .

mcdeimon commented 5 years ago

Hi. Now I see that my server has a hard time loading the records from the administrator.

VidYen commented 5 years ago

@mcdeimon

Hey I just rehauled the whole point log system. Mostly on the admin side (and some minor stuff on the public side)

Let me know if it works for you or at least better if you could test the github version please. For the user log, I have to work with the nonce issue and the public log function doesn't have that built in. So I am going to just leave it at the last 50 for now.

Before it was everyone on the list so I guess its better than nothing.

I need to add search and other logs. However, I think I am going to save that for 1.8.2

I think 1.8.1 actually does a lot of good fixes so I may push it up after more testing.