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

Try out SDK v6.0.0-alpha1 in Firely Server #3009

Open
alexzautke opened this issue Jan 14, 2025 · 1 comment
Open

Try out SDK v6.0.0-alpha1 in Firely Server #3009

alexzautke opened this issue Jan 14, 2025 · 1 comment
Assignees

Comments

@alexzautke
Copy link
Member

No description provided.

@andrzejskowronski
Copy link

Blocking:
Both legacy Validator and the new one refer to 5.11, and won't work with 6.0.0. Either/both need to be adjusted for 6.0.0 to proceed.

Changes that were needed:

  • ElementNodeExtensions.ToScopedNode changed return to IScopedNode, so any code that relied on concrete type needed to be adjusted, either with upcasting, or wrapping in ScopedNode, but that has some side-effects (losing fullUrl, Parent)
  • Base no longer implements IReadOnlyDictionary<string, object> causing collection initializers to fail. Example below no longer compiles with error that the type does not implement IEnumerable
    • new Parameters {{"rawResult", new FhirString(json)}};
  • SearchParamType moved namespaces, which led to conflicts with FS copy of the enum
  • SearchParamDefinition moved namespace, needed to fix references
  • In this test, the second ShortPath is equal to Location now as it fails on this behaviour - if it would rather check for IShortPathGenerator it would work as expected
  • SourceNode.Valued and SourceNode.Resource changed nullability, so some calls needed !
  • FhirTypeAttribute.IsResource removed, some logic needed to be adjusted for that
  • Base.NamedChildren removed, replaced with EnumerateElements for the same behaviour
  • FhirJsonSerializationSettings gone from ITypedElement.ToJson

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

No branches or pull requests

3 participants