Skip to content

Comments

DOC-5653 Update ZDM diagrams#234

Merged
brian-r-fisher merged 10 commits intomainfrom
DOC-5653
Feb 20, 2026
Merged

DOC-5653 Update ZDM diagrams#234
brian-r-fisher merged 10 commits intomainfrom
DOC-5653

Conversation

@brian-r-fisher
Copy link
Contributor

@brian-r-fisher brian-r-fisher commented Feb 17, 2026

Updates ZDM diagrams to remove specific databases, adds official IBM colors from https://www.ibm.com/design/language/color/

Preview: About the ZDM process

@brian-r-fisher brian-r-fisher self-assigned this Feb 17, 2026
@plpesvc-ds

This comment has been minimized.

@plpesvc-ds

This comment has been minimized.

@plpesvc-ds

This comment has been minimized.

@plpesvc-ds

This comment has been minimized.

@plpesvc-ds

This comment has been minimized.

Copy link
Contributor

@aimurphy aimurphy left a comment

Choose a reason for hiding this comment

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

The ovals don't look quite as polished as the ones in the original images, but the information is all there.

Optional: I think the arrows and "reads/writes" could be black. I'm a little concerned the yellow and orange don't have high enough contrast with the background.

I defer to @eric-schneider regarding use of Figma.

@plpesvc-ds

This comment has been minimized.

@brian-r-fisher
Copy link
Contributor Author

The ovals don't look quite as polished as the ones in the original images, but the information is all there.

Optional: I think the arrows and "reads/writes" could be black. I'm a little concerned the yellow and orange don't have high enough contrast with the background.

I defer to @eric-schneider regarding use of Figma.

Good point, thanks. I updated the arrows and "reads/writes" to be black.

@eric-schneider
Copy link
Contributor

@brian-r-fisher I'll see what I can do in Figma and get back to you. Thank you for providing the link to the IBM color palette 😄

@eric-schneider
Copy link
Contributor

@brian-r-fisher @aimurphy What do you think about this?

migration-phase4ra9

@brian-r-fisher
Copy link
Contributor Author

@brian-r-fisher @aimurphy What do you think about this?

migration-phase4ra9

LGTM

@aimurphy
Copy link
Contributor

@brian-r-fisher @aimurphy What do you think about this?

LGTM as well

@plpesvc-ds

This comment has been minimized.

@eric-schneider
Copy link
Contributor

@aimurphy would you be able to re-review? I made some slight modifications to the text in the diagrams. I originally only planned on lower-casing "Origin" and "Target", but made the executive decision to go with "origin cluster" and "target cluster" since that's what's most prevalent in the docs.

@aimurphy
Copy link
Contributor

aimurphy commented Feb 20, 2026

About the ZDM process

~I don't see a change on the preview 👀 ~ I see it now. I had to reload.

However, in the Phase 2 image, can you change DSBulk Migrator to just DSBulk? DSBulk Migrator no longer exists.

@plpesvc-ds
Copy link

plpesvc-ds commented Feb 20, 2026

Build successful! ✅
Deploying draft.
Deploy successful! View draft

@brian-r-fisher brian-r-fisher merged commit 59352cc into main Feb 20, 2026
1 check passed
@brian-r-fisher brian-r-fisher deleted the DOC-5653 branch February 20, 2026 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants