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

Add i18n localized strings to images component #586

Conversation

rak-phillip
Copy link
Contributor

This addresses a small piece of tech debt by adding i18n localized strings to the images component.

#585

@rak-phillip rak-phillip self-assigned this Sep 3, 2021
Copy link
Contributor

@mook-as mook-as left a comment

Choose a reason for hiding this comment

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

Yay!

We'll need to figure out how to manage resources copied from dashboard though; previously, we could just replace the translations files wholesale with the upstream copy. Now we'll need to figure out something for merging.

@@ -25,44 +25,44 @@
<Card :show-highlight-border="false" :show-actions="false">
<template #title>
<div class="type-title">
<h3>Image Acquisition</h3>
<h3>{{ t('images.manager.title') }}</h3>
Copy link
Contributor

Choose a reason for hiding this comment

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

I believe we can use v-text here. I'm actually not sure if that's clearer, though. This applies to similar constructs as well.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It really comes down to style preference. I think we prefer the mustache syntax in dashboard, but I'm willing to stick with whatever we prefer for Rancher Desktop.

Copy link
Contributor

Choose a reason for hiding this comment

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

Oh I have no idea what we prefer; just went through Vue docs at one point… :D

@rak-phillip
Copy link
Contributor Author

Yay!

We'll need to figure out how to manage resources copied from dashboard though; previously, we could just replace the translations files wholesale with the upstream copy. Now we'll need to figure out something for merging.

We might want to look at strategies for keeping the two separate. I've created #589 so we can track this progress

@rak-phillip rak-phillip merged commit e6ca291 into rancher-sandbox:main Sep 7, 2021
@rak-phillip rak-phillip deleted the feature/images-component-translations branch September 7, 2021 17:55
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