cube-js/cube

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

Open

#8.711 geöffnet am 16. Sept. 2024

Auf GitHub ansehen
 (2 Kommentare) (2 Reaktionen) (0 zugewiesene Personen)Rust (1.965 Forks)batch import
api:resthelp wanted

Repository-Metriken

Stars
 (19.563 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 5T 16h) (138 gemergte PRs in 30 T)

Beschreibung

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!

Contributor Guide

Research-Richtung
Finde die Funktion transformMetaExtended in packages/cubejs api gateway/src/helpers/transformMetaExtended.ts. Füge das Feld 'sql table' zum zurückgegebenen Objekt hinzu, wahrscheinlich neben 'sql'. Aktualisiere die Unit Tests im selben Verzeichnis, um zu überprüfen, ob das neue Feld in der Antwort enthalten ist. Stelle sicher, dass das Feld sowohl mit als auch ohne den Abfrageparameter 'extended' vorhanden ist.
Tech Stack
typescript
Domain
backend
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
TypeScriptGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
85