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

【確認待ち】各ブロックにdescriptionが指定されているかチェック、ない場合は設定する #571

Merged
merged 26 commits into from
Aug 3, 2021

Conversation

shimotmk
Copy link
Contributor

@shimotmk shimotmk commented Jun 11, 2021

こちらお時間あるときに確認お願いします🙏

issue
#536

description一覧
#536 (comment)

descriptionチェックリスト
#536 (comment)

環境

  • WordPress 5.8

@shimotmk shimotmk requested a review from miminari June 16, 2021 01:54
@shimotmk shimotmk marked this pull request as ready for review June 16, 2021 01:55
@miminari
Copy link
Contributor

@shimotmk PRありがとうございます!
こちらですが、5.8からdescriptionの翻訳テキストもblock.jsonに移行する
WordPress/gutenberg#30293
https://ja.wordpress.org/team/handbook/block-editor/reference-guides/block-api/block-metadata/
ので、そんなに急ぐ変更ではないため、ちょうどいいので、その形式で実装できたらと思います。

ただ、WordPress 5.8β2の段階でコアの翻訳体制が整っていないので、もう少し待ってから実装していきたいと思います。

詳しくは
https://meta.trac.wordpress.org/ticket/5737
wp-cli/i18n-command#253

(おそらく、wp-cliのバージョンをアップする必要があります。)

@shimotmk
Copy link
Contributor Author

shimotmk commented Jun 22, 2021

@miminari
ありがとうございます!
なるほど、了解しました
コアの体制整ってから実装します!

■自分用メモ
.wp-env.jsonでcoreのバージョンを指定
"core": "https://wordpress.org/wordpress-5.8-beta1.zip",
block.jsonにdescriptionを書いたら反映された
翻訳はコアのバージョンがリリースされててから実装
https://github.com/wp-cli/i18n-command/pull/253/files

@shimotmk shimotmk changed the title 各ブロックにdescriptionが指定されているかチェック、ない場合は設定する 【WIP】各ブロックにdescriptionが指定されているかチェック、ない場合は設定する Jun 22, 2021
@miminari
Copy link
Contributor

miminari commented Jun 29, 2021

@shimotmk 調査ありがとうございますー!
wp-envを自分のローカルだけで書き換えるのには、

.wp-env.override.json
https://ja.wordpress.org/team/handbook/block-editor/reference-guides/packages/packages-env/#.wp-env.override.json

を利用するのがオススメです。

"core": "WordPress/WordPress#master",

だとNightlyが適用されるのでより良いかと。
翻訳問題がFIXしたぽいので、開発の継続をお願いしますー

@shimotmk shimotmk changed the title 【WIP】各ブロックにdescriptionが指定されているかチェック、ない場合は設定する 【確認待ち】各ブロックにdescriptionが指定されているかチェック、ない場合は設定する Jul 7, 2021
@shimotmk
Copy link
Contributor Author

shimotmk commented Jul 7, 2021

WordPress5.8のdescription反映しました
問題がないか確認願いします🙏

@shimotmk
Copy link
Contributor Author

@miminari
すみません
本日の終礼でおっしゃっていたのはこちらのdiscussionをresolve conversationにするということであっていますでしょうか?

Copy link
Contributor

@miminari miminari left a comment

Choose a reason for hiding this comment

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

@miminari
すみません
本日の終礼でおっしゃっていたのはこちらのdiscussionをresolve conversationにするということであっていますでしょうか?

説明不足ですみません、

ここに、後方互換処理を入れないといけないはず

なのでその対応についてのお話でした!

src/blocks/bundle.js Show resolved Hide resolved
src/blocks/bundle.js Show resolved Hide resolved
@miminari
Copy link
Contributor

miminari commented Aug 3, 2021

@shimotmk 5.7でもブロックを読み込むような後方互換を入れました。ご確認ください。

@shimotmk
Copy link
Contributor Author

shimotmk commented Aug 3, 2021

@miminari
ありがとうございます!
5.7、5.8でdescription表示されていること確認出来ました!

Copy link
Contributor

@miminari miminari left a comment

Choose a reason for hiding this comment

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

確認ありがとうございます。マージしますねー

@miminari miminari merged commit a0336b0 into master Aug 3, 2021
@shimotmk shimotmk deleted the fix/block-settings-description-add branch August 5, 2021 00:38
@miminari miminari removed their assignment Jan 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants