Skip to content

Conversation

@joaquincasal
Copy link
Collaborator

Purpose

Adding a new step in the Dialog of the app to compare the valus for the source and one target locale.

Approach

This PR:

  • Changes the way we update entries, from using sdk.entry to sdk.cma.entry. This will allow to update entries referenced in a future PR.
  • Adds a new PreviewStep to the Dialog that shows values for the source and target locales.
  • Shows a diff between the source and target values, using colored backgrounds.

For the rich text fields, the diff is shown keeping the formatting from the field, which required separate logic for the diff.

Testing steps

Automated tests were updated and added.

Breaking Changes

N/A

Dependencies and/or References

Four libraries were added for the diff logic

Deployment

N/A

Screen.Recording.2026-02-03.at.09.48.34.mov

@joaquincasal joaquincasal requested a review from a team as a code owner February 3, 2026 12:50
@wiz-inc-38d59fb8d7
Copy link

wiz-inc-38d59fb8d7 bot commented Feb 3, 2026

Wiz Scan Summary

Scanner Findings
Vulnerability Finding Vulnerabilities 2 Low
Data Finding Sensitive Data -
Secret Finding Secrets -
IaC Misconfiguration IaC Misconfigurations -
SAST Finding SAST Findings 1 Medium
Software Management Finding Software Management Findings -
Total 1 Medium 2 Low

View scan details in Wiz

To detect these findings earlier in the dev lifecycle, try using Wiz Code VS Code Extension.

result: UpdateResult;
}

const ConfirmationStep = ({ result }: ConfirmationStepProps) => {
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This looks quite different from the designs because it will change in the next PR, it will require the updateEntry function to return different information (info from other updated entries)

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.

2 participants