Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[Workflow] [WIP] #28874 add style to workflow dumps #28933

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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

alexislefebvre
Copy link
Contributor

@alexislefebvre alexislefebvre commented Oct 20, 2018

Q A
Branch? master
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets #28874
License MIT
Doc PR TODO

Fetch data from MetadataStore #26092 in order to add colors to the dumps.

This is a proof of concept to show that it's possible to define the color of a Transition, but I need some help to define the best way to provide a simple and powerful way to configure the outputs of the workflow dumps. Please add your thoughts about this.

I started a discussion about the different options here: #28874 (comment)


Background

Here is how we can define the colors of a transition, a Red arrow and a Blue label :

"created" -[#Red]-> "submitted": "<font color=Blue>search"

So we have to inject some data in the PUML file.


Default output and output with this PR:

complex-state-machine-marking_old complex-state-machine-marking

(the t2 transition is blue and the t1 transition is red)

@alexislefebvre alexislefebvre changed the title [Workflow] [WIP] #28874 add colors to workflow dumps [Workflow] [WIP] #28874 add style to workflow dumps Oct 20, 2018
@nicolas-grekas nicolas-grekas added this to the next milestone Oct 20, 2018
@lyrixx
Copy link
Member

lyrixx commented Nov 8, 2018

Hello, Thanks for this PR. As we are not clear about the configuration (see your issue #28874)
I'm going to close this PR. This is just for cleaning. No worry. (I'm sorry BTW, this never fun to close a PR). I will be very happy to merge your PR once it's ready. So please, let's comment on the issue first, find a nice configuration, and then we could reopen the PR

@lyrixx lyrixx closed this Nov 8, 2018
@alexislefebvre alexislefebvre deleted the 28874-add-colors-to-workflow-dumps branch November 8, 2018 15:51
@stof stof removed this from the next milestone Nov 23, 2018
fabpot added a commit that referenced this pull request Mar 19, 2019
This PR was squashed before being merged into the 4.3-dev branch (closes #29538).

Discussion
----------

[Workflow] Add colors to workflow dumps

Fixes #28874

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | no
| New feature?  | yes
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #28874, replaces #28933
| License       | MIT
| Doc PR        | TODO, requires symfony/symfony-docs#9476

Fetch data with the `MetadataStore` from #26092 in order to add colors to the dumps.

Example of configuration:

```yaml
            transitions:
                submit:
                    from: start
                    to: travis
                    metadata:
                        title: transition submit title
                        dump_style:
                            label: 'My custom label'
                            arrow_color: '#0088FF'
                            label_color: 'Red'
```

This code was developed as a bundle, examples can be found on its repository: https://github.com/alexislefebvre/SymfonyWorkflowStyleBundle

Commits
-------

60ad109 [Workflow] Add colors to workflow dumps
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants