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.

Sign up for GitHub

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

Jump to bottom

Implement "LoadbyDefault" on a model property #140

Closed
1 task done
jansenbe opened this issue Sep 14, 2020 · 3 comments
Closed
1 task done

Implement "LoadbyDefault" on a model property #140

jansenbe opened this issue Sep 14, 2020 · 3 comments
Assignees
@jansenbe
Labels
area: framework ⚙ Changes in the SDK core framework code

Comments

@jansenbe
Copy link
Contributor

Category

  • Feature request

Describe the feature

The key property today will always be loaded, but it makes sense to make this model more flexible and allow other properties to also be loaded by default. An example use case would be the SharePointUser model: if we would always load the AadObjectId property then that would save a roundtrip when transforming the SharePoint user into a graph user
@jansenbe jansenbe added this to the Beta 1 milestone Sep 14, 2020
@jansenbe jansenbe self-assigned this Sep 14, 2020
@jansenbe
Copy link
Contributor Author

Also links to #38 as this is a metadata extension

@jansenbe jansenbe modified the milestones: Beta 1, Beta 2 Dec 7, 2020
@jansenbe
Copy link
Contributor Author

jansenbe commented Dec 7, 2020

Not urgent for being able to ship GA so moving to backlog

@jansenbe jansenbe removed this from the Beta 2 milestone Dec 7, 2020
@jansenbe jansenbe added area: framework ⚙ Changes in the SDK core framework code and removed enhancement labels Dec 8, 2020
@jansenbe
Copy link
Contributor Author

Complexity outweighs the benefits here, closing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jansenbe jansenbe

Labels
area: framework ⚙ Changes in the SDK core framework code
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jansenbe