Skip to content
This repository has been archived by the owner on Apr 20, 2018. It is now read-only.

Remove ref doc will broke the parent router #82

Closed
crapthings opened this issue Jun 14, 2016 · 2 comments
Closed

Remove ref doc will broke the parent router #82

crapthings opened this issue Jun 14, 2016 · 2 comments

Comments

@crapthings
Copy link

let's have service A and service B
when A has many B

A: { childrenIds: [1, 2, 3] }

then you use populate hook on A like

const findPatents = hooks.populate('children', {
  service: 'B',
  field: 'childrenIds'
})

you remove any B will cause A router "404 not found"

is this consider a bug ?

@daffl
Copy link
Member

daffl commented Jun 14, 2016

I'm not sure what else it should do? It could probably throw a more specific error like Error populating 'children': <error message here>.

@daffl
Copy link
Member

daffl commented Apr 16, 2017

This should no be handled in the new populate hook in feathers-hooks-common.

@daffl daffl closed this as completed Apr 16, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants