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

Devtools 保存機能 #41

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Devtools 保存機能 #41

wants to merge 7 commits into from

Conversation

kobkaz
Copy link
Contributor

@kobkaz kobkaz commented Dec 20, 2023

保存内容の具体的なフォーマットは未定

テレメトリ名の横の〇マークをクリックで保存する/しないをトグル
sharedworker起動後初回の保存ではディレクトリ選択画面が開き、以降はそのディレクトリを基準として ${テレメトリ名}/${保存開始時間}.log に保存される
blobは${テレメトリ名}/blob_data/${保存時間}.datに保存

@kobkaz kobkaz self-assigned this Dec 20, 2023
@kobkaz kobkaz force-pushed the devtools_recording branch 4 times, most recently from 49407b2 to e68ef27 Compare December 21, 2023 01:20
@kobkaz
Copy link
Contributor Author

kobkaz commented Dec 21, 2023

出力仕様など未定ですが、一旦この辺りでフロントエンドに詳しい人に見てもらいたいと思います @KOBA789

@kobkaz kobkaz force-pushed the devtools_recording branch from e68ef27 to 27be099 Compare January 12, 2024 07:06
@kobkaz kobkaz requested a review from sksat as a code owner January 12, 2024 07:06
@kobkaz kobkaz force-pushed the devtools_recording branch from 27be099 to b4abc37 Compare January 12, 2024 07:24
@KOBA789
Copy link
Member

KOBA789 commented Feb 9, 2024

これ完全に私が止めちゃってますよねすみません。見ます。

@KOBA789
Copy link
Member

KOBA789 commented Feb 14, 2024

  • UI は改良の余地がありそう
    • 右側のテレメビューの上に Navbar 追加して、そこに保存開始ボタン付けるのがいいだろうか
  • すべての JSON が改行などなしにすべて繋がっている?
    • これは見間違えかも。JSON がデカすぎて目視ではわからなかった……

@kobkaz kobkaz force-pushed the devtools_recording branch from b4abc37 to bfe1baa Compare February 19, 2024 06:13
@kobkaz
Copy link
Contributor Author

kobkaz commented Feb 19, 2024

mainにリベース
短い保存フォーマットを使用
レコード間に改行を入れた

@kobkaz
Copy link
Contributor Author

kobkaz commented Feb 19, 2024

右側のテレメビューの上に Navbar 追加して

機能だけ実装したけれど、どうすれば見た目がまともになるかが全くわかりません

@KOBA789
Copy link
Member

KOBA789 commented Feb 19, 2024

このブランチに向かって UI の変更の PR 作ってみます。より良くなるかは自信ないですが……

@kobkaz kobkaz force-pushed the devtools_recording branch from b3ed474 to 9447f1c Compare February 20, 2024 07:14
@kobkaz kobkaz force-pushed the devtools_recording branch 3 times, most recently from b7dcfdb to ee94baa Compare February 21, 2024 00:38
@KOBA789 KOBA789 added this to the v1.1.0 milestone Mar 12, 2024
@kobkaz kobkaz force-pushed the devtools_recording branch from 540db5f to 42b261f Compare April 12, 2024 06:22
@kobkaz kobkaz requested a review from KOBA789 as a code owner April 12, 2024 06:22
@kobkaz kobkaz force-pushed the devtools_recording branch from aaff426 to e8fb69b Compare April 12, 2024 06:43
@kobkaz
Copy link
Contributor Author

kobkaz commented Apr 12, 2024

blobについては当初と実装方針が変わったため、blob保存機能は一旦削除する

@kobkaz
Copy link
Contributor Author

kobkaz commented Apr 12, 2024

これもうマージしてしまいたいので、UIだけ見てもらえますか @KOBA789

@kobkaz kobkaz force-pushed the devtools_recording branch from 6b5cc35 to 7419fb2 Compare April 12, 2024 07:40
Copy link
Member

@KOBA789 KOBA789 left a comment

Choose a reason for hiding this comment

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

0.7 のリリースが詰まっているのでマージは保留します。
もしかしたら 0.7 に滑り込ませるかもしれません。

@kobkaz kobkaz changed the title (WIP) Devtools 保存機能 Devtools 保存機能 Apr 18, 2024
@KOBA789
Copy link
Member

KOBA789 commented May 22, 2024

@kobkaz こちら、マージ作業進めちゃっていいですか?

@kobkaz
Copy link
Contributor Author

kobkaz commented May 22, 2024

いいと思います

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.

2 participants