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

【お知らせ記事待ち】設定のインポート・エクスポートができるようにしたい #1539

Closed
mthaichi opened this issue Jan 12, 2023 · 13 comments
Assignees

Comments

@mthaichi
Copy link
Contributor

mthaichi commented Jan 12, 2023

カスタム書式やカスタムブロックスタイル・ブロックマネージャーなどの便利な設定が増えてきたのですが、細々設定したものを他のサイトでも使いたいという場合、手動で設定しなければなりませんよね?

吹き出しブロック設定は画像があるのでちょっとむずかしいですかねー(画像をBase64でエンコードするとか?)

カスタム書式やカスタムブロックスタイル・ブロックマネージャーの設定だけでもインポート・エクスポートできたらいいなと思いますし、「カスタムブロックスタイルだけエクスポートする」みたいな選択ができたらなお良いと思いますが、いかがでしょうか。

@kurudrive
Copy link
Member

@mthaichi いいですね!
吹き出しは除外で良いのではないでしょうか?
サイトによって登場人物はかわると思いますし。

@kurudrive kurudrive changed the title 設定のインポート・エクスポートができるようにしたい 設定のインポート・エクスポートができるようにしたい - まずは仕様を考える Jan 24, 2023
@kurudrive kurudrive changed the title 設定のインポート・エクスポートができるようにしたい - まずは仕様を考える 【仕様検討】設定のインポート・エクスポートができるようにしたい Jan 24, 2023
@kurudrive
Copy link
Member

kurudrive commented Jan 24, 2023

これは json でエクスポート、インポートが妥当ですかねぇ。

  • エクスポートボタン
  • ファイル選択ボタン(インポート用)
  • インポートボタン

を追加するみたいなのを想定しました。
そのほか高度な機能ってなくていいですよね?

ただ、そもそも、その前段階としてまだ option 値の処理のリファクタリングの途中ですよね?

そうなると option 値の扱いのリファクタリングと 6.2 対応の後になると思うから優先度はまだ低い感じかなと思いました。

@mthaichi @shimotmk @drill-lancer @doshimaf
何かご意見・提案などありましたら募集中ですー。

@mthaichi
Copy link
Contributor Author

これは json でエクスポート、インポートが妥当ですかねぇ。

  • エクスポートボタン
  • ファイル選択ボタン(インポート用)
  • インポートボタン

を追加するみたいなのを想定しました。 そのほか高度な機能ってなくていいですよね?

ただ、そもそも、その前段階としてまだ option 値の処理のリファクタリングの途中ですよね?

そうなると option 値の扱いのリファクタリングと 6.2 対応の後になると思うから優先度はまだ低い感じかなと思いました。

@mthaichi @shimotmk @drill-lancer @doshimaf 何かご意見・提案などありましたら募集中ですー。

自分は、ACFのインポート・エクスポート機能の感じをイメージしていました。
エクスポートでどの項目をエクスポートするか選べるといいなと。「カスタム書式」「カスタムブロックスタイル」「ブロックマネージャー」のチェックボックスがあり、「エクスポート」押下でダウンロード。
インポートはファイルを選んで「インポート」を押すだけ。

↓ACFのインポート・エクスポート画面
image

@shimotmk
Copy link
Contributor

@mthaichi
これトライしてみても良いですか?

@mthaichi
Copy link
Contributor Author

@mthaichi これトライしてみても良いですか?

@shimotmk もちろんです。ありがとうございます。m(__)m

@shimotmk
Copy link
Contributor

この機能を作ってみました。
#1667

エクスポートする項目とインポートする項目を選べるようにしました。
吹き出しブロック、カスタム書式設定、カスタムブロックスタイル設定はインポート方法も追加しました。

仕様などご意見ある方いただければと思います。

@mthaichi
Copy link
Contributor Author

@shimotmk 一点だけ、気になった点を。

エクスポート時は、現在の画面の「入力値」がJSONに反映されているようですが、「保存値」をエクスポートしたほうがいいと思います。

インポートの時はそのまま保存され「保存値」になるので、エクスポートも「保存値」をJSON化したほうがいいのではという理由です。また、インポート&エクスポートは「変更を保存」ボタンより下にあるので、設定フォームとはまったく別の枠組みという捉え方をします。設定フォームの内容によって、JSON化される内容が設定フォームの入力値に左右されないほうがよいかと思いましたが、いかがでしょう。

あとはワタシ的には、イメージ通りのもので、すばらしいと思います。実装ありがとうございます!

@shimotmk
Copy link
Contributor

@mthaichi
ありがとうございます。

エクスポート時は、現在の画面の「入力値」がJSONに反映されているようですが、「保存値」をエクスポートしたほうがいいと思います。

これは自分も結構迷ったのですが、保存値の方が良さそうですよね。
保存値と入力値が異なっていたらエクスポートは出来ないようなどにした方が良さそうですね。

また、インポート&エクスポートは「変更を保存」ボタンより下にあるので、設定フォームとはまったく別の枠組みという捉え方をします。設定フォームの内容によって、JSON化される内容が設定フォームの入力値に左右されないほうがよいかと思いましたが、いかがでしょう

これもエクスポート時のJSON化の話ですか?
設定フォームの内容 = 入力値 という意味ですか?

@mthaichi
Copy link
Contributor Author

mthaichi commented May 19, 2023

これもエクスポート時のJSON化の話ですか?
設定フォームの内容 = 入力値 という意味ですか?

そうです。インポート&エクスポートのエリアは、VK Blocksの各種設定フォーム(=入力値)の外側にある形なので、JSONの内容が設定フォーム(=入力値)の内容に左右されるのはよろしくないのかなと感じました。

@mthaichi
Copy link
Contributor Author

これは自分も結構迷ったのですが、保存値の方が良さそうですよね。
保存値と入力値が異なっていたらエクスポートは出来ないようなどにした方が良さそうですね。

それでもいいと思いますし、「変更した設定が保存されていないようです。変更前の設定内容をエクスポートします」とお断りした上で、エクスポートしちゃっても良いかなと思います。

@shimotmk
Copy link
Contributor

shimotmk commented May 19, 2023

@mthaichi
ありがとうございます。
保存値と入力値が異なっていたらエクスポートは出来ないようは難しかったので、「変更した設定が保存されていないようです。変更前の設定内容をエクスポートします」とお断りを追加しておきました!#1667

「保存値」をエクスポート対象にも修正しました🙏

@shimotmk
Copy link
Contributor

shimotmk commented Jun 1, 2023

特に何もなさそうなのでそのままプルリクにします〜

@sysbird sysbird changed the title 【仕様検討】設定のインポート・エクスポートができるようにしたい 【リリース待ち】設定のインポート・エクスポートができるようにしたい Jun 6, 2023
@kurudrive kurudrive changed the title 【リリース待ち】設定のインポート・エクスポートができるようにしたい 【お知らせ記事待ち】設定のインポート・エクスポートができるようにしたい Jun 23, 2023
@kaorock72
Copy link

@kurudrive お知らせ公開いたしました!
https://www.vektor-inc.co.jp/product-update/vk-blocks-1-58-0-import-export/

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

No branches or pull requests

6 participants