Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(populate): take child param into account when populating #1085

Merged

Conversation

danielo515
Copy link
Contributor

@danielo515 danielo515 commented Apr 25, 2021

Description

Check List

If not relevant to pull request, check off as complete

  • All tests passing
  • Docs updated with any changes or examples if applicable
  • Added tests to ensure new feature(s) work properly

Relevant Issues

@prescottprue prescottprue changed the title fix: take child param into account when populating fix(populate): take child param into account when populating May 9, 2021
@prescottprue prescottprue changed the base branch from master to v3.11.0 November 23, 2021 05:05
@prescottprue prescottprue merged commit 35db11a into prescottprue:v3.11.0 Nov 23, 2021
@prescottprue prescottprue mentioned this pull request Nov 23, 2021
3 tasks
prescottprue added a commit that referenced this pull request Nov 23, 2021
* feat(auth): support passwordless sign-in (#857, #1064) - @komachi
* fix(populate): take child param into account when populating (#1083, #1085) - @danielo515
* fix(docs): correctly redirect when auth does not exist in the routing example doc - @JonathanPorta
* chore(deps): bump ws in /examples/complete/typescript (#1148)
* chore(deps): bump color-string in /examples/complete/typescript (#1147)
* chore(deps): bump dns-packet in /examples/complete/typescript (#1110)
* chore(deps): bump dns-packet in /examples/complete/firestore (#1109)
* chore(deps): bump dns-packet in /examples/complete/simple (#1108) 

Co-authored-by: Anton Nesterov <[email protected]>
Co-authored-by: Daniel Rodríguez Rivero <[email protected]>
Co-authored-by: Jonathan Porta <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants