Skip to content

Updating JA translation of ASA content #478

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

Merged
merged 12 commits into from
Oct 10, 2022

Conversation

paulwallace-okta
Copy link
Contributor

@paulwallace-okta paulwallace-okta commented Sep 30, 2022

@netlify
Copy link

netlify bot commented Sep 30, 2022

Deploy Preview for extraordinary-horse-71260e ready!

Name Link
🔨 Latest commit 7726175
🔍 Latest deploy log https://app.netlify.com/sites/extraordinary-horse-71260e/deploys/634482d0210a9500082d6692
😎 Deploy Preview https://deploy-preview-478--extraordinary-horse-71260e.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

In the latest translation update, the "title" attribute for
the Okta Help Center logo was translated, which broke a test.
It is preferable to have this string appear in JA rather than EN,
thus refactoring the test.
The top level menubar has links to product doc and release notes.
Previously these pointed to JA pages, applying a fix here to
avoid a regression of these URLs pointing to EN links.
Copy link

Choose a reason for hiding this comment

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

looks good, I added some comments to point to jiras to automate all these issues. And I left one question to be sure what happened

<div class="navbar">
<div class="dropdown" data-mc-conditions="MultiProdPubish.DocMenu"> <button class="dropbtn">ドキュメント <i class="fa fa-caret-down"></i></button> <div class="dropdown-content">
<div><a href="https://help.okta.com/ja-jp/Content/Topics/ReleaseNotes/okta-relnotes.htm">リリースノート</a> </div>
<div><a href="https://help.okta.com/ja-jp/Content/index.htm">製品ドキュメント</a> </div>

Choose a reason for hiding this comment

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

this changes we should automate, we have https://oktainc.atlassian.net/browse/OKTA-540003 for it

</div>
<div>
<h2><a name="assign-users-or-groups-to-ASA"></a>ユーザーまたはグループを<span class="mc-variable okta-feature-names.ScaleFT variable">アドバンストサーバーアクセス</span><br />に割り当てる</h2>
<p>前の手順を終えると、属性の構成が完了し、使用を開始できます。<span class="mc-variable okta-feature-names.ScaleFT variable">アドバンストサーバーアクセス</span>に割り当てられたユーザーには、これらの属性が割り当てられます。「<a href="https://help.okta.com/okta_help.htm?type=oie&amp;id=csh-assign-apps" target="_blank">アプリケーションのユーザーへの割り当て</a>」をご覧ください。</p>
<p>前の手順を終えると、属性の構成が完了し、使用を開始できます。<span class="mc-variable okta-feature-names.ScaleFT variable">アドバンストサーバーアクセス</span>に割り当てられたユーザーには、これらの属性が割り当てられます。「<a href="../../../../../../okta_help.htm?locale=ja-JP&amp;type=oie&amp;id=csh-assign-apps" target="_blank">アプリケーションのユーザーへの割り当て</a>」をご覧ください。</p>

Choose a reason for hiding this comment

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

This changes should be automated, https://oktainc.atlassian.net/browse/OKTA-538298 should take care of this

@@ -328,6 +328,8 @@ <h1 class="relnotes">未翻訳のドキュメント</h1>
</div>
<!-- ********************** Scripts ************************************* -->
<!-- Foundation -->
<script src="../../Resources/Scripts/js/ignore-dompurify.js">

Choose a reason for hiding this comment

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

https://oktainc.atlassian.net/browse/OKTA-538286 should take care of automating this (if needed)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

In this case, I believe we added the ignore-dompurify.js script to the EN doc after we sent this to translation. So, it'll be automatically included in future rounds.

@@ -195,25 +195,25 @@ <h1>アドバンストサーバーアクセス</h1>
<p class="tile-subtitle"><span class="mc-variable okta-feature-names.ScaleFT variable">アドバンストサーバーアクセス</span>のチームをセットアップし、<span class="mc-variable okta-variables.ProductName variable">Okta</span>からユーザーとグループをプッシュします。</p>
</div>
<div>
<p class="tile-title">サーバーエージェント</p>
<p class="tile-title"><a href="sftd.htm">サーバーエージェント</a></p>

Choose a reason for hiding this comment

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

What happened here? Did XTM removed the <a> elements or the version we sent did not have it?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We had missing links in the tiles at the time that we sent the project out (the issue was in source EN, it's been fixed).

@paulwallace-okta paulwallace-okta merged commit e4791cd into gh-pages Oct 10, 2022
@paulwallace-okta paulwallace-okta deleted the l10n-JA-asa-2022.08.08 branch November 30, 2022 23: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