-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
chore: convert relationships to use identifiers #7470
Conversation
Asset Size Report for 31dbbb7 IE11 Builds 🛑 The size of the library EmberData has increased by +1.4 KB (+428.0 B compressed) which exceeds the failure threshold of 75 bytes.Warnings
Changeset
Full Asset Analysis (IE11)
Modern Builds 🛑 The size of the library EmberData has increased by +1.35 KB (+483.0 B compressed) which exceeds the failure threshold of 75 bytes.Warnings
Changeset
Full Asset Analysis (Modern)
Modern Builds (No Rollup) 🛑 The size of the library EmberData has increased by +1.71 KB (+423.0 B compressed) which exceeds the failure threshold of 75 bytes.WarningsChangeset
Full Asset Analysis (Modern)
|
38c9547
to
31dbbb7
Compare
010804a
to
b62c4ff
Compare
c7e83c2
to
1acd5b5
Compare
1acd5b5
to
62d849f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not the most knowledgeable person to review this PR but I did learn a few things! Nothing stands out as out of place and you certainly have an idea of where this work is heading. Great job and appreciate you walking me through this PR!!
packages/record-data/addon/-private/relationships/state/belongs-to.ts
Outdated
Show resolved
Hide resolved
Keep up the great work @runspired |
6fdd826
to
2bd374f
Compare
Revisits and replaces #6845