The pathParse function in RescriptReactRouter.res currently treats single slash and double slash URLs identically, which causes routing conflicts and prevents proper URL differentiation.
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 65/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 停滞
- 技術スタック
- react
- 領域
- frontend
調査の方向性
RescriptReactRouter.res の pathParse 関数から始め、スラッシュで区切られたパスに対する split と filter のロジックを調べてください。/analytics-transaction と //analytics-transaction の動作を確認し、その結果のリストが先頭の空のセグメントを二重スラッシュの URL の場合にのみ保持することを確認してください。
索引モデルが issue の本文から書いたものです。
説明
Current Behavior
// Both URLs return the same parsed result
pathParse("/analytics-transaction") → list{"analytics-transaction"}
pathParse("//analytics-transaction") → list{"analytics-transaction"} // Problem!
Expected Behavior
// URLs should parse differently to enable proper routing
pathParse("/analytics-transaction") → list{"analytics-transaction"}
pathParse("//analytics-transaction") → list{"", "analytics-transaction"} // Should preserve leading empty string
Impact
- Cannot differentiate between /path and //path in routing logic
- Causes "Page Not Found" errors for valid double slash URLs
- Breaks routing patterns that rely on double slash prefixes
Proposed Solution
Modify the filter logic to preserve the first empty string while removing others:
// Current problematic code
raw->Js.String2.split("/")->Js.Array2.filter(item => item->Js.String2.length != 0)->arrayToList
// Proposed fix
let splitArray = raw->Js.String2.split("/")
let filteredArray = []
splitArray->Js.Array2.forEachi((item, index) => {
if item->Js.String2.length != 0 || index == 0 {
filteredArray->Js.Array2.push(item)->ignore
}
})
filteredArray->arrayToList
- 主要言語
- ReScript
- スター
- 517
- フォーク
- 45
- PR マージ指標
- 30日以内にマージされた PR はありません
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
rescript-lang/rescript-react のほかの issue
-
難易度 3/5 1〜2日 初心者へのやさしさ 48/100
rescript-lang/rescript-react#152 · リアクション 3 件 ·
-
難易度 5/5 1週間以上 初心者へのやさしさ 45/100
rescript-lang/rescript-react#147 · コメント 2 件 ·
-
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
rescript-lang/rescript-react#111 · コメント 3 件 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 25/100
rescript-lang/rescript-react#105 ·
-
Scroll Restoration オープン
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
rescript-lang/rescript-react#104 · リアクション 1 件 ·
rescript-lang/rescript-react の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
lobehub/lobe-icons#422 · コメント 1 件 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
-
community first-timers-only good first issue hacktoberfest help wanted low hanging fruit up-for-grabs
難易度 1/5 1時間未満 初心者へのやさしさ 82/100
-
www.guideflow.com オープンN: AdGuard Browser Extension P4: Low T: Social Widget
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
AdguardTeam/AdguardFilters#242250 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100