-
-
Notifications
You must be signed in to change notification settings - Fork 498
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
Update doc regarding async relationships #1817
base: master
Are you sure you want to change the base?
Conversation
Adding doc hints for async relationships as discussed on discord: https://discord.com/channels/480462759797063690/480777444203429888/971550923124002916
Mention default value Co-authored-by: Joao Da Silva Marly <[email protected]>
@runspired @snewcomer would you like to review this ember data related PR for the Guides? |
Just a reminder, this PR could use a reviewer from the Ember Data team. I plan to merge this at the end of this week. @PaulRosset I left some comments with small wording updates for clarity/precise vocabulary. Let me know if you have any questions about why I picked those terms to use. |
P.S. Thanks for this PR! |
Co-authored-by: Jen Weber <[email protected]>
Co-authored-by: Jen Weber <[email protected]>
Co-authored-by: Jen Weber <[email protected]>
Co-authored-by: Jen Weber <[email protected]>
@jenweber Thanks for your suggestions! I updated the file accordingly :) |
@jenweber Have u had time to go through the recent changes? thanks :) |
Adding doc hints for async relationships as discussed on Discord.