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

InventoryViewer: フォルダのお気に入り登録の登録場所がおかしい #17

Closed
blhsrwznrghfzpr opened this issue Feb 22, 2022 · 3 comments
Labels
bug Something isn't working

Comments

@blhsrwznrghfzpr
Copy link
Collaborator

InventoryViewerでフォルダを開いているときに星アイコンでお気に入り登録すると、
お気に入りされる場所が正しく登録されない

image

画像の上2つはフォルダからお気に入り登録したもの

  • JP Tools: Essential Toolsからお気に入り登録。開くと何も表示されない
  • Brushes: Essential Toolsからお気に入り登録。開くとEssential Toolsが表示される
  • rhenium Public: topからインベントリリンクを貼り付けてお気に入り登録。正しく表示される。
@rheniumNV
Copy link
Collaborator

報告ありがとうございます!

原因が分かりました。
・ディレクトリの場合、移動時にパスと名前を結合するべきところで名前が入っていなかった
・リンクの場合、そもそもパスが全く違うため移動できなかった

ディレクトリは名前を追加すれば問題なし。
リンクの場合はRecordIdを使い、/inventory/v1/viewrではなく/inventory/v1/linkに飛ばせばよい。
しかし、お気に入りの情報にrecordTypeを入れていなかったため見分けがつかないという問題あり、悩んでいます。

さらに、ディレクトリもリンクもRecordIdがあるのでどちらもinventory/v1/linkに飛ばすという選択もありますが、そもそもlinkは一度レコード情報を取得する必要があり遅いです。

色々とごちゃごちゃになってしまったので、一度壊してお気に入り登録時にリンク先のパスの情報まで保存するように作りなおすのもありかなと考えているところでした。

@rheniumNV
Copy link
Collaborator

動かないままはよくないので一旦、全てlinkに飛ばして動くようにしました。
#21

@rheniumNV rheniumNV added the bug Something isn't working label Feb 23, 2022
@rheniumNV
Copy link
Collaborator

バグという意味では解決したのでここは閉じて、別イシューを残すことにしました。
#22

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants