Azure / Azure/data-api-builder

Update how nextLink value is retrieved

Abierto
#2,901 0 comentarios 0 reacciones 0 asignados Ver en GitHub
Lenguaje dominante
C#
Estrellas
1.5k
Forks
370
Merge medio
3 d 17 h
PR fusionados (30 d)
8

Descripción

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

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

Comienza revisando la discusión del pull request referenciado y localizando el código que recupera el valor nextLink/after. Confirma el manejo actual de los corchetes y, después, verifica que los valores entre corchetes se extraigan, mientras que los valores sin corchetes conserven toda la información relevante.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
csharp
Área
api
Tipo de issue
Error
Dificultad
2/5
Tiempo estimado
1-3 horas
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
45/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.