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

Change Bundle.entry from List to IList. #2742

Open
mmsmits opened this issue Mar 25, 2024 · 1 comment
Open

Change Bundle.entry from List to IList. #2742

mmsmits opened this issue Mar 25, 2024 · 1 comment
Labels
breaking change This issue/commit causes a breaking change, and requires a major version upgrade

Comments

@mmsmits
Copy link
Member

mmsmits commented Mar 25, 2024

It will be easier this way to write performant helper methods this way, to query for specific types of resources in a Bundle for example.

This is a breaking change, so we should schedule this when we are planning the next major release.

@mmsmits mmsmits added enhancement breaking change This issue/commit causes a breaking change, and requires a major version upgrade labels Mar 25, 2024
@Kasdejong Kasdejong added the SDK-6 label Oct 2, 2024
@Kasdejong Kasdejong modified the milestone: SDK 6 Oct 9, 2024
@Kasdejong Kasdejong removed the SDK-6 label Oct 9, 2024
@mmsmits
Copy link
Member Author

mmsmits commented Oct 10, 2024

Maybe:

  • Only a couple of lists like Bundle.entry (not all lists). To implement an observable collection.

https://learn.microsoft.com/en-us/dotnet/api/system.collections.objectmodel.observablecollection-1?view=net-8.0
or
https://learn.microsoft.com/en-us/dotnet/api/microsoft.visualstudio.extensibility.ui.observablelist-1?view=vs-extensibility

Other candidates:

  • Composition.entry
  • List.entry
  • Group (?).
  • Maybe more?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking change This issue/commit causes a breaking change, and requires a major version upgrade
Projects
None yet
Development

No branches or pull requests

2 participants