NebulaLogger icon indicating copy to clipboard operation
NebulaLogger copied to clipboard

Summer '22 release - v4.8.0

Open jongpie opened this issue 1 year ago • 2 comments

  • Bumped all metadata to API v55.0 (Summer '22 release)

  • Closed #335 by adding new method Logger.ignoreOrigin(Type apexType) - this allows plugins/external code to specify Apex classes to ignore when parsing each log entry's stack trace/origin

  • Fixed #339 by cleaning up the Admin.profile that's currently included with the recipes metadata - the Admin.profile will eventually be replaced with a new permission set, but for now, this should make the metadata deployable

  • Fixed #340 by adding some extra checks for empty lists in builder's methods stripInnaccessible() and getJson()

  • Fixed #346 by updating test stack traces for Aura & LWC to reflect the latest content + updating the logic used for parsing the component's stack trace

  • Fixed a null object exception that could occur in LogMessage when the message template is null (something I noticed when discussing #342)

  • Fixed #331 by correcting some links in README for the docs site

jongpie avatar Jul 29 '22 13:07 jongpie

@jamessimone if you have a few minutes, I'd love to get your thoughts on one part of this PR - there's a new method called Logger.ignoreOrigin(Type apexType) to skip certain classes in the log entry's stack trace/origin fields (something apex-rollup's Nebula Logger plugin could use). We had briefly discussed it last month, but wanted to see if you have any concerns with this approach now that I've got a PR. Everything else in the PR is mostly related to upgrading from API v54.0 --> v55.0, and a few minor bugfixes, so feel free to ignore the other 100+ files that have changed 😳 .

jongpie avatar Jul 29 '22 15:07 jongpie

Codecov Report

Merging #343 (caa3bbf) into main (8d00cf5) will decrease coverage by 0.35%. The diff coverage is 85.71%.

:exclamation: Current head caa3bbf differs from pull request most recent head 67797d9. Consider uploading reports for the commit 67797d9 to get more accurate results

@@            Coverage Diff             @@
##             main     #343      +/-   ##
==========================================
- Coverage   95.50%   95.14%   -0.36%     
==========================================
  Files          49       51       +2     
  Lines        4826     4989     +163     
  Branches       94       96       +2     
==========================================
+ Hits         4609     4747     +138     
- Misses        214      238      +24     
- Partials        3        4       +1     
Flag Coverage Δ
Apex 95.68% <82.24%> (-0.38%) :arrow_down:
LWC 91.86% <91.80%> (-0.01%) :arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...in/log-management/classes/LogEntryEventHandler.cls 93.04% <79.48%> (-1.98%) :arrow_down:
...log-management/classes/LogBatchPurgeController.cls 83.07% <83.07%> (ø)
.../log-management/lwc/logBatchPurge/logBatchPurge.js 91.80% <91.80%> (ø)
...main/log-management/classes/LogEntryTagHandler.cls 100.00% <100.00%> (ø)
...ain/logger-engine/classes/LogEntryEventBuilder.cls 96.80% <0.00%> (-0.22%) :arrow_down:
...ore/main/logger-engine/classes/ComponentLogger.cls 98.23% <0.00%> (-0.02%) :arrow_down:
...ger/core/main/logger-engine/classes/FlowLogger.cls 100.00% <0.00%> (ø)
...ger/core/main/logger-engine/classes/LogMessage.cls 100.00% <0.00%> (ø)
...-logger/core/main/logger-engine/classes/Logger.cls 95.16% <0.00%> (+0.04%) :arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

codecov[bot] avatar Aug 24 '22 14:08 codecov[bot]