Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork9.6k
[Workflow] Add a profiler#51204
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
dcf97da
to1e9fc3f
CompareThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
I'm totally a bad UI designer, so I found this on the web, and I adapted it a bit.
Feel free to replace it with something better
1e9fc3f
toe90147a
Comparesrc/Symfony/Component/Workflow/DataCollector/WorkflowDataCollector.php OutdatedShow resolvedHide resolved
Uh oh!
There was an error while loading.Please reload this page.
src/Symfony/Bundle/WebProfilerBundle/Resources/views/Collector/workflow.html.twig OutdatedShow resolvedHide resolved
Uh oh!
There was an error while loading.Please reload this page.
e90147a
todf34d97
Comparedf34d97
to773f468
Compare773f468
todf292cb
CompareThis panel does not collect any info related to the request. It only collect info about the configuration of the project, which will be the same for all requests. In the past, we removed some panels like that. |
lyrixx commentedAug 1, 2023 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
I thought about doing everything at runtime (I mean without all the collector stuff). But I changed my mind because the workflow can evolve between 2 requests. And we collect already some information about listener, doctrine entities. API-P does the same with the resources, and so on And this panel isreally useful. I'm working on a new projet, with a huge workflow, and it helped me a lot to build the configuration. And I plan to add more information to it, like the list of all event listeners. This features has been requested in#50976 So I wouldbe very happy to get this merged, it'll be a game changer for everyone using the workflow component |
We already one such panel: "configuration" |
lyrixx commentedAug 1, 2023 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Note: In future version, this panal will also display more things
So it could not fit in the configuration panel It will be something like the security panel : some configuration + some request related content |
i also find it useful even if not linked to request/response flow |
Thank you@lyrixx. |
Uh oh!
There was an error while loading.Please reload this page.
@lyrixx I used your WorkflowExtension to dump Workflow builded on the fly (in Controller), but I see some blocks is rendered on grey background ![]() How you fixed this? I used the same code on frontend (mermaid.js and the same ver of bootstrap.min.css) |
I'm not sure to understand your issue. Could could open a github discussion to talk about that? Thanks |
Withthis project lyrixx/SFLive-Paris2016-Workflow
workflow.webm