Azure / Azure/data-api-builder

Update how nextLink value is retrieved

オープン
#2,901 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
C#
スター
1.5k
フォーク
370
平均マージ
3日 22時間
マージ済み PR(30日)
9

説明

Currently the nextLink/after value is retrieved by removing the brackets and saving the contents. However, it is done by assuming that the brackets are in the first and last positions of the string. We need to update it to ensure that we do not delete any relevant information in case the brackets are not there.

Solutions:
1) Using `IndexOf` and `LastIndexOf` functions to check if the brackets exist and use their positions to delete them with regex (Similar to the way that it is done now)
2) Using only regex to only use the information found inside the brackets if they exist.

This task is made in reference to the following comment:
https://github.com/Azure/data-api-builder/pull/2893#discussion_r2413840792

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

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

調査の方向性

まず、参照されている pull request のディスカッションを確認し、nextLink/after 値を取得するコードを見つけます。現在の角括弧の処理を確認し、そのうえで、角括弧で囲まれた値が抽出され、角括弧のない値では関連する情報がすべて保持されることを検証します。

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

評価

技術スタック
csharp
領域
api
issue の種類
バグ
難易度
2/5
見積もり時間
1〜3時間
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
45/100

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

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