Vincit/objection.js

Missing typings for QueryBuilder.tableRef

Open

#2364 aperta il 16 mar 2023

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)JavaScript (663 fork)batch import
good first issuetypings

Metriche repository

Star
 (7146 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

QueryBuilder has a convenient method tableRef which returns the alias for the model the query builder operates against:

https://github.com/Vincit/objection.js/blob/1b74f756c8afd3bab7964ac3aea7438b95edce44/lib/queryBuilder/QueryBuilder.js#L87-L89

However it's not exposed in typings and not mentioned in the documentation. Can we add it?

On a side note, its implementation clearly contradicts the typings for tableRefFor:

https://github.com/Vincit/objection.js/blob/1b74f756c8afd3bab7964ac3aea7438b95edce44/typings/objection/index.d.ts#L615-L617

the method is supposed to accept a model class, but the actual implementation passes a string.

Guida contributor

Direzione di ricerca
Esamina il file typings/objection/index.d.ts per aggiungere la firma del metodo `tableRef` mancante per QueryBuilder.
Tech stack
javascripttypescript
Dominio
backend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
TypeScriptGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
85