cube-js/cube

Meta API (`/meta`): Add the `sql_table` field to the response

Open

#8 711 ouverte le 16 sept. 2024

Voir sur GitHub
 (2 commentaires) (2 réactions) (0 assignés)Rust (19 563 stars) (1 965 forks)batch import
api:resthelp wanted

Description

The /v1/meta API endpoint does not include the sql_table field. Even with the ?extended query param the sql_table field does not exist, even though sql does. It would also be great to have this included unless there is a good reason not to.

I believe it's the following function that needs to be adapted along with the unit tests: https://github.com/cube-js/cube/blob/e8d81f2e04ce61215b1185d9adc4320233bc34da/packages/cubejs-api-gateway/src/helpers/transformMetaExtended.ts#L37

I'm happy to open a pull request for the same!

Guide contributeur

Stack technique
typescriptnodejsrest api
Domaine
backendapi
Type d'issue
feature
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
under 1 hour
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
fresh
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
TypeScriptREST API experienceUnit testing
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
90
Direction de recherche
The issue is to add the `sql table` field to the `/v1/meta` API response. The specific file to modify is `packages/cubejs api gateway/src/helpers/transformMetaExtended.ts`, specifically the function around line 37. You will need to add the field and update the corresponding unit tests. No pull request is currently linked, so this issue is available for a first time contributor.