関数とパラメーターに説明のリンク追加 #534
関数とパラメーターに説明のリンク追加 #534
Conversation
tsuchiyama-araya
requested review from
emuemuJP,
itutu-tienday and
milesAraya
frontend/src/components/Workspace/FlowChart/RightDrawer/index.tsx
Outdated
Show resolved
Hide resolved
frontend/src/components/Workspace/FlowChart/RightDrawer/index.tsx
Outdated
Show resolved
Hide resolved
frontend/src/components/Workspace/FlowChart/RightDrawer/AlgorithmParamForm.tsx
Outdated
Show resolved
Hide resolved
frontend/src/components/Workspace/FlowChart/RightDrawer/AlgorithmParamForm.tsx
Outdated
Show resolved
Hide resolved
frontend/src/components/Workspace/FlowChart/RightDrawer/index.tsx
Outdated
Show resolved
Hide resolved
|
こちらのPRがCloseしたら、こちらのPRもCloseします。 |
| @@ -26,6 +29,8 @@ export const AlgorithmParamForm = memo(function AlgorithmParamForm() { | |||
| selectAlgorithmParamsKeyList(nodeId), | |||
| arrayEqualityFn, | |||
| ) | |||
| const formattedAlgoName = algoName.toLowerCase().replace(/_/g, "-") | |||
| const parameterUrl = `https://optinist.readthedocs.io/en/latest/specifications/algorithm_nodes.html#${formattedAlgoName}` | |||
There was a problem hiding this comment.
リンク成功のテストを行って下さい
※少し確認してみたところ、リンク切れが多数ありそうです（リンク先にコンテンツはあるようです）
There was a problem hiding this comment.
こちら対応できました。
There was a problem hiding this comment.
すべて確認していませんが、複数件のリンク切れがまだあるようです。
→ lccd_cell_detection, cnmf_multisession, ...
すべてのNode Paramのリンク成功を、再チェックしてください。
There was a problem hiding this comment.
@itutu-tienday
こちら対応と確認できました。
お手数ですが、ご確認お願いいたします。
There was a problem hiding this comment.
まだリンク切れがあるようです
roi_fluo_from_hdf5
※次回以降の同様の対応では、チェック完了のエビデンスをシェアしてください。
There was a problem hiding this comment.
そちらは、間違ってないかと思います。
一番下のページなので、もうそれ以上いきません。
There was a problem hiding this comment.
There was a problem hiding this comment.
なるほど、roi_fluo_from_hdf5ですね。
こちらは、まだ記載されていないのですが、こちらのブランチで対応したほうが良いのでしょうか？
また、DocsとParamsリンクのリリースは同時で行いますか？
There was a problem hiding this comment.
ドキュメントは用意しないのですか？
Node作成者・ドキュメント編集者間で、調整してください。
frontend/src/components/Workspace/FlowChart/RightDrawer/AlgorithmParamForm.tsx
Show resolved
Hide resolved
| @@ -26,6 +29,8 @@ export const AlgorithmParamForm = memo(function AlgorithmParamForm() { | |||
| selectAlgorithmParamsKeyList(nodeId), | |||
| arrayEqualityFn, | |||
| ) | |||
| const formattedAlgoName = algoName.toLowerCase().replace(/_/g, "-") | |||
| const parameterUrl = `https://optinist.readthedocs.io/en/latest/specifications/algorithm_nodes.html#${formattedAlgoName}` | |||
There was a problem hiding this comment.
【内容】
#510
パラメーターのタイトルと関数の説明にリンクで説明に遷移させました。
ドキュメント先のリンク：