search

Laravel-Backpack / demo

A working demo of Laravel with all Backpack packages installed.
http://backpackforlaravel.com
Other
329 stars 165 forks source link

Added Activity Log package #569

Closed promatik closed 1 year ago

request-info[bot] commented 1 year ago

Hi there!

Could you please provide us with more info about this? Looks like you skipped the title/body.

Thank you!

-- Justin Case The Backpack Robot

karandatwani92 commented 1 year ago

Here is my review of the activity log:

Majorly I find it the way I expected. I agree with the above suggestion to trim down the names. Good Job @promatik🎉

BUGs

  1. I faced an error on the activity_changes column.
  2. Causer & Subject filter stops working after reset.

Enhancements:

  1. Filters should provide null option; causedBy() & performedOn() are optional
  2. Event filter should be select2_multiple
  3. Add a table(causer, event, description & time) example with a custom view in the show operation.
  4. Create an activity_log column displaying table(causer, event, description & time).
  5. Causer should be wrapped with a link; same as the subject.
  6. Show only events related to the selected modal type in event filter. (Ajax)
tabacitu commented 1 year ago

Thanks @karandatwani92 !

I faced an error on the activity_changes column.

I believe I have already fixed that today, in https://github.com/Laravel-Backpack/activity-log/pull/15