JKHeadley/rest-hapi

Add option to populate subdocs

Open

#104 ouverte le 18 avr. 2018

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)JavaScript (1 181 stars) (156 forks)batch import
enhancementhelp wanted

Description

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

Guide contributeur

Stack technique
javascriptnodejsmongodb
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.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
half day
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
mostly clear
Prérequis
Mongoose populateNode.jsREST API design
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
35
Direction de recherche
First, review the linked comment in issue #102 and the existing `populateEmbeddedDocs` function in the rest hapi codebase (likely in a helper file). Understand how embedding is currently handled. Then, propose a syntax to differentiate between deep populate (e.g., `subdoc:pets`) and simple populate (e.g., `subdoc.pets`). Consider updating the documentation and adding tests for the new option.