burkybang / SuiteAdvanced-Feedback

NetSuite Advanced Feedback
https://burkybang.com/SuiteAdvanced
3 stars 0 forks source link

In script executions logs some json gets beautified some does not. #91

Closed eduardRsult closed 1 year ago

eduardRsult commented 1 year ago

Describe the bug A clear and concise description of what the bug is.

Answer: In script executions logs some json gets beautified some does not. Previously only JOSN that was inclomplete (cutoff by log details character limit) was not beautified. Now it just parses some and some not.


How to reproduce _Describe steps to reproduce the behavior. Run a map reduce that creates many logs with json. Some longer some shorter.


Expected behavior A clear and concise description of what you expected to happen. All json beautified


Screenshots I have 2 (arbitrary) situations

  1. only system logs
  2. all logs

I made screenshots for each of them in 3 conditions

  1. with this plugin (SA) (black theme) While the other plugin is turned off (full refresh of page)
  2. with the other plugin that conflicts with SuiteAdvanced with SA turned off (full refresh of page)
  3. vanilla with both plugins turned off. (full refresh of page)

Additional context I am pretty sure this worked better before. Nice that the scriptIds are clickable to the record type. Would be great to have an easy way to link to an instance like discussed on Linkedin

Screenshot 2023-05-31 at 11 48 17 Screenshot 2023-05-31 at 11 56 03 Screenshot 2023-05-31 at 11 55 46 Screenshot 2023-05-31 at 11 49 08 Screenshot 2023-05-31 at 11 48 56 Screenshot 2023-05-31 at 11 48 32

burkybang commented 1 year ago

@eduardRsult Are you using .stringify(value, null, 2) before logging the objects to the log? If so, that could be the issue.

eduardRsult commented 1 year ago

Yes sometimes the module I use spits that out. Or it is added instead of the default toString() method to error objects since they are sometimes not only thrown but also saved.

eduardRsult commented 1 year ago

Is there a way you could handle this like the other plugin? It's valid JSON after all.

burkybang commented 1 year ago

Yeah, I'll see what I can do

burkybang commented 1 year ago

I found a solution. It'll be in the next release (ETA a few days).

burkybang commented 1 year ago

I submitted a new release to the Chrome Web Store. It'll be available whenever Google approves it.

burkybang commented 1 year ago

Update: It's been approved.