acore-cms icon indicating copy to clipboard operation
acore-cms copied to clipboard

Feature soap logs view

Open nicoaravena opened this issue 3 years ago • 12 comments

Goal

List some SOAP commands executed from de CMS in favor to have more information in case of some purchase fails or something wrong happens.

Features

  • New view Soap Logs

Proposed changes

  • Add a new option to the executeCommand function to allow log it and add an order_id if applies.

Code

AC-CMS ADMIN

  • [x] Base service file/handler
  • [x] New table wp_acore_soap_logs
  • [x] New view Soap Logs available just for WP admins
  • [x] Filter by username and order id
  • [x] Allow pagination

nicoaravena avatar Sep 20 '22 03:09 nicoaravena

this feature is very good, thanks a lot! 🚀

Helias avatar Sep 20 '22 08:09 Helias

I just push fixes and changes.

nicoaravena avatar Sep 22 '22 00:09 nicoaravena

except that raf code that has been removed, the rest seems ok to me hope that code was dead code and it does not affect RAF, if you are sure about it we can merge this PR

thanks for this logging feature, it will be useful 🚀

Helias avatar Sep 22 '22 08:09 Helias

If you can check if buy an item from the store works fine, then I can merge it, but I want to have a double check.

nicoaravena avatar Sep 22 '22 14:09 nicoaravena

tested char rename service and it worked but sending an item does not work, I receive an mail in the mailbox but empty, with no items :/

Helias avatar Sep 22 '22 23:09 Helias

@Helias did the log show anything related?

nicoaravena avatar Sep 23 '22 15:09 nicoaravena

it says nothing image

Helias avatar Sep 28 '22 14:09 Helias

I receive an empty mail

image

Helias avatar Sep 28 '22 14:09 Helias

I fixed both problems

nicoaravena avatar Sep 30 '22 00:09 nicoaravena

if you confirm that you tested it we can merge this PR.

So, hope you tested at least one service with sending item via mailbox and one service like char rename

Helias avatar Oct 01 '22 09:10 Helias

@Helias I'm not been able to retest the commands, do to some troubles in my pc with the core. Sorry for the late response.

nicoaravena avatar Oct 23 '22 22:10 nicoaravena

Don't worry, If I will have time I'll test it

Helias avatar Oct 24 '22 07:10 Helias