[1.0.0-alpha2] Better logs

Created on 8 June 2025, 28 days ago

Problem/Motivation

Logs panel is a devel tool today, with technical information. Let's transform it to an user friendly tool.

Proposed resolution

Updated data log meesage

Today, we have just @instance_id has been updated

It would be nice to have @instance has been updated: @diff where:

  • @instance is the label ( or label with summary) instead of the instance ID.
  • @diff is the difference between instance data before after. How can we get this diff and make sure it is pretty printed and understandable by users? Maybe we can do this only for components at first.

Don't show up instance IDS

Today we have this kind of : 684607c5347e9 token has been attached to 6845fb8953f5c's prefix slot at position 0

It would be nice to:

  • use label ( or label with summary) instead of the instance ID
  • don't mention the term "slot" and don't tell the posiiton

Example: "Globe icon has been attached to horizontal card's prefix"

Add username in state history logs

To prepare the data for the real-time collaboration feature.

📌 Task
Status

Active

Version

1.0

Component

State Manager

Created by

🇫🇷France pdureau Paris

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @pdureau
  • First commit to issue fork.
  • Merge request !22Issue #3529074 by goz, pdureau: Better logs → (Merged) created by goz
  • 🇫🇷France goz

    I can't figure out what i can load from instance id.
    Btw, i think it should be more accurate to not display label instance, but path change from to (with labels).

    Not : Globe icon has been attached to horizontal card's prefix
    But :

    • Globe icon has been attached to Grid Row 1 col (Bootstrap) > horizontal card's prefix
    • Grid Row 1 col (Bootstrap) > Globe icon has been attached to Grid Row 1 col (Bootstrap) > Grid Row col (Bootstrap) > horizontal card's prefix

    Anyway, i still missing how to get label from ids or path

  • Pipeline finished with Failed
    3 days ago
    Total: 339s
    #538950
  • 🇫🇷France goz

    It would be nice to have @instance has been updated: @diff

    Is way more complicated, may be we could move it in another issue like discussed during meeting ?

  • Pipeline finished with Failed
    3 days ago
    Total: 147s
    #539057
  • 🇫🇷France goz

    PHPStan fails on code which is not part of this MR.

  • Pipeline finished with Failed
    3 days ago
    Total: 139s
    #539062
  • 🇫🇷France pdureau Paris

    Is way more complicated, may be we could move it in another issue like discussed during meeting ?

    OK, I will create a dedicated issue for the beta phase, so no rush.

    So:

    • Updated data message: Can we remove the extra "source" word. Example: "Alert source has been updated" > "Alert has been updated"
    • Updated data message (when triggered by ThirdParty): Can we remove the extra "island" word and use the island's label instead of plugin ID? Example: "Icon: heroicons:arrow-down-on-square-stack has been updated by Styles"
    • Attached data message: OK
    • Removed data message: OK
    • Add username in state history logs: OK
    • Remove first col: OK

    Thanks Goz. It is way more friendly like that!

    I give it back to you for my 2 little feedback and the ones from Jean in the MR

  • Pipeline finished with Failed
    about 11 hours ago
    Total: 138s
    #540304
  • 🇫🇷France pdureau Paris

    Jean, i don't undesrtand the phpstan error:

      Line   src/StateManager/StateManager.php                               
     ------ ---------------------------------------------------------------- 
      418    Offset 'summary' might not exist on array{label:                
             Drupal\Core\StringTranslation\TranslatableMarkup}|array{label:  
             string, summary: string}.                                       
     ------ --------------------------------
    

    Is the error in the interface declaration or in the interface implementation?

Production build 0.71.5 2024