Skip to content
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

feat(core): allow users to override pipeline graph positions #7141

Merged
merged 1 commit into from
Jun 26, 2019
Merged

feat(core): allow users to override pipeline graph positions #7141

merged 1 commit into from
Jun 26, 2019

Conversation

anotherchrisberry
Copy link
Contributor

We've had a few requests over the years from users who want more control in the rendering of the pipeline graph. We work very hard to minimize a) the overlapping lines and b) the length of the lines.

Sometimes users want to override this. This PR provide that ability by adding another field to the stage JSON.

Not sure about the field name (does graphRowOverride make any sense) or about how to document this. I feel like it should be documented beyond "in the code" and "tribal knowledge". Feelings?

Copy link
Contributor

@louisjimenez louisjimenez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would this be something the user sets via the edit json in deck? Would it make sense to be documented under the user guide for editing a pipeline as json?

Copy link
Contributor

@gcomstock gcomstock left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

exceptional null guards

@anotherchrisberry
Copy link
Contributor Author

@louisjimenez yeah, that is a good idea. I will add some documentation there!

@anotherchrisberry anotherchrisberry merged commit 342087c into spinnaker:master Jun 26, 2019
anotherchrisberry added a commit that referenced this pull request Jun 26, 2019
…94 (#7150)

* chore(core): Bump version to 0.0.377

d5425b4 refactor(core): virtualize execution rendering (#7140)
342087c feat(core): allow users to override pipeline graph positions (#7141)
fb58d70 fix(core): do not send a cloud provider on v2 search calls (#7142)
5d6d9aa chore(core): clarify clone stage help text (#7131)
745f0a1 fix(core): Display latest template in pipeline template list (#7145)
456172b fix(webhooks): addresses issue 3450 - introduce a delay before polling webhook (#7144)
7238d1d feat(core): Enable new artifacts workflow in bakeManifest (#7138)
45e5aa3 feat(artifacts): find multiple artifacts from single execution (#7139)
dae73da fix(core): provide key for repeating param JSX elements (#7136)
bbc1d06 fix(core): filter falsy error messages from errors object on tasks (#7135)
4782808 refactor(core): Reactify overrideTimeout (#7126)
0a3bd68 feat(core/presentation): Always call onBlur in Checklist to "mark as touched" (#7134)
cdd6f23 chore(package): Just Update Prettier™
7e464a9 fix(amazon): Support SpEL in advanced capacity (#7124)
08e9506 chore(deck): Update to Typescript 3.4
fa515dc fix(core): do not stretch provider logos in selection modal (#7128)

* chore(docker): Bump version to 0.0.42

8358010 fix(docker): Allow auto-switch to manual entry when refreshing images (#7120)

* chore(amazon): Bump version to 0.0.194

cdd6f23 chore(package): Just Update Prettier™
7e464a9 fix(amazon): Support SpEL in advanced capacity (#7124)
08e9506 chore(deck): Update to Typescript 3.4
@anotherchrisberry anotherchrisberry deleted the graph-row-override branch January 8, 2020 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants