JKHeadley/rest-hapi

Add option to populate subdocs

Open

#104 aperta il 18 apr 2018

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)JavaScript (156 fork)batch import
enhancementhelp wanted

Metriche repository

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

Descrizione

Via this comment:

It would be great if there were some way to specify an embed of 'subdoc.pets' that made populateEmbeddedDocs call .populate({ path: 'subdoc.pets' }) instead of .populate({ path: 'subdoc', populate: { path: 'pets' } }).

Need to develop syntax to specify between the two, ex: subdoc.pets vs subdoc:pets

Guida contributor

Direzione di ricerca
Innanzitutto, rivedi il commento collegato nell'issue #102 e la funzione esistente `populateEmbeddedDocs` nella codebase di rest hapi (probabilmente in un file helper). Comprendi come è attualmente gestita l'incorporazione. Poi, proponi una sintassi per differenziare tra il popolamento profondo (es. `subdoc:pets`) e il popolamento semplice (es. `subdoc.pets`). Considera di aggiornare la documentazione e aggiungere test per la nuova opzione.
Tech stack
javascriptnodejsmongodb
Dominio
backendapi
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
Mongoose populateNode.jsREST API design
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
35