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.

Sign up for GitHub

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

Jump to bottom

fix(components, protocol-designer): Update translations according to initial pre-DQA copy/style pass #17333

Merged
merged 3 commits into from
Jan 23, 2025
Merged

fix(components, protocol-designer): Update translations according to initial pre-DQA copy/style pass #17333

merged 3 commits into from
Jan 23, 2025

Conversation

ncdiehl11
Copy link
Collaborator

@ncdiehl11 ncdiehl11 commented Jan 23, 2025 •
edited
Loading

Overview

Update translations according to initial pre-DQA copy/style pass

Closes AUTH-1283

Test Plan and Hands on Testing

inspect copy

Changelog

  • update copy and label padding according to designs
  • add tooltip to current initialization settings in absorbance reader tools

Review requests

  • see test plan

Risk assessment

low

@ncdiehl11 ncdiehl11 self-assigned this Jan 23, 2025
@ncdiehl11 ncdiehl11 requested review from koji and jerader January 23, 2025 15:32
@ncdiehl11 ncdiehl11 marked this pull request as ready for review January 23, 2025 15:32
@ncdiehl11 ncdiehl11 requested review from a team as code owners January 23, 2025 15:32
@ncdiehl11 ncdiehl11 removed request for a team January 23, 2025 15:32
@ncdiehl11 ncdiehl11 merged commit c7bb1b3 into edge Jan 23, 2025
58 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@koji koji koji approved these changes

@jerader jerader Awaiting requested review from jerader

Assignees

@ncdiehl11 ncdiehl11

Labels
authorship
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ncdiehl11 @koji