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

Ember Data notes 01 november 2017 #211

Merged
merged 1 commit into from
Feb 15, 2018
Merged

Conversation

hjdivad
Copy link
Member

@hjdivad hjdivad commented Oct 26, 2017

No description provided.


## Attendees

## Agenda
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reply here with agenda items

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Follow up on solving various use cases unload via addon

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

in particular

  • addon for transitive-unload
  • addon for soft-delete

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Discuss testing API's:

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Discuss emberjs/data#5230 and whether/how to support multi-table inheritance.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Potentially discuss allowing unique indexes against simple (non-compound) attributes

@hjdivad
Copy link
Member Author

hjdivad commented Oct 26, 2017

This is the format I'd suggested in this week's meeting @pangratz @bmac @runspired @dgeb

@workmanw
Copy link

workmanw commented Nov 1, 2017

👋 -- Obviously I'm not a team member and don't have a say in what is/isn't discussed, but I've read these for years to keep up to date on what's happening in ember, ember-cli, and ember-data.

I'd really like to see the outcome of a discussion around all the unloadRecord issues that began with 2.12. I'm one of at least 20 people who are still stuck on older versions and it'd be really great to at least know these things are being discussed, if not actively worked on.

@runspired
Copy link

The outcome is roughly that we are going to investigate enabling the soft-delete and purge use-cases via addon. We're stuck in a hard place because we can't go back as that would reintroduce the bugs this change fixed, but nor can we fix forward as the issue is that there are three fundamentally different use-cases involved in how folks were using unloadRecord for better or for worse.

@workmanw
Copy link

workmanw commented Nov 1, 2017

I'd be interested to hear more, but this is probably not the right venue to ask these questions.

@locks
Copy link
Collaborator

locks commented Nov 2, 2017

@hoIIer
Copy link

hoIIer commented Nov 3, 2017

hello, curious if there was/is any discussion on the natural-key issue for models?

@runspired
Copy link

@erichonkanen no, but I believe this will fix that: emberjs/data#5249

@hjdivad
Copy link
Member Author

hjdivad commented Nov 8, 2017

@locks locks merged commit d293760 into emberjs:master Feb 15, 2018
@hjdivad hjdivad deleted the hjdivad/ed-20171101 branch February 15, 2018 01:48
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.

8 participants