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

Update intro-to-storybook/ja #765

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

UHAsikakutou
Copy link

No description provided.

Copy link
Collaborator

@jonniebigodes jonniebigodes left a comment

Choose a reason for hiding this comment

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

@UHAsikakutou appreciate the time you took to put together this pull request and add the much-deserved updates to the translation. 🙏 ! I've checked and left a couple of suggestions for you to look into when you have a moment. Also, if you're okay with it, as you are already working on updating the translation, can you check #768 and include those changes here? For context they're small adjustments to some of the examples.

Let me know once you've addressed it, and I'll be more than glad to take another pass at it so that we can get someone in the community to review it.

Will that work for you?

Have a fantastic day.

Stay safe

loading: false,
export const Empty = {
args: {
// argsによってストーリーを形成します。
Copy link
Collaborator

Choose a reason for hiding this comment

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

@UHAsikakutou, I'd appreciate it if you could revert the comments in the tutorial back to English. We're aware that translating into a native language brings the translation closer to home (pardon the bad pun), but until we have a better way of handling it, we prefer to keep them as they are.

steps:
- uses: actions/checkout@v1
- uses: actions/checkout@v3
Copy link
Collaborator

Choose a reason for hiding this comment

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

@UHAsikakutou can you replace the checkout action to be "actions/checkout@v4" instead?

- run: yarn
#👇 Adds Chromatic as a step in the workflow
#👇 Chromaticをワークフローのステップとして追加
Copy link
Collaborator

Choose a reason for hiding this comment

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

Also, if you're ok with it, the chromaui action can be adjusted to be "chromaui/action@latest".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants