patternfly / patternfly/react-data-view

New exports from `DataViewTh` hide type exported from `DataViewTable`

オープン 初心者向け
#662 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

主要言語
TypeScript
スター
4
フォーク
21
30日以内にマージされた PR はありません

説明

Version 6.5 adds new exports from DataviewTh, but this hides the DataViewTh type already exported from DataViewTable [1]. This results in build errors due to the type/const mismatch.

error TS2749: 'DataViewTh' refers to a value, but is being used as a type here. Did you mean 'typeof DataViewTh'?

[1] https://github.com/patternfly/react-data-view/commit/16bc87dd31b9a0802effc7e7349cad8d9b1f985f#diff-146876574627e7fb43f6b4c0af8c98e89b44660c64fc8522d032afea5cd797e1

Jira Issue: PF-4240

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

参照されているコミットの export の変更を調査し、DataViewTable からの既存の DataViewTh 型 export と比較します。DataViewTh を引き続き型として使用できるように名前の衝突を解消し、その後 TypeScript のビルドまたは関連するチェックを実行して、TS2749 エラーが解消されたことを確認します。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
react, typescript
領域
frontend
issue の種類
バグ
難易度
2/5
見積もり時間
1〜3時間
活発さ
静か
明瞭さ
おおむね明確
初心者へのやさしさ
68/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。