Coder Social home page Coder Social logo

Comments (5)

rsmayda avatar rsmayda commented on July 21, 2024 2

Hey @lyndoh after further review, I have no modifications or qualms with this approach. This makes a great deal of sense and we would support this sort of PR. As you mentioned this would be a backwards breaking change so I believe the order of operations would be:

  1. Update interface (bumping major version) - removing the interface method
    • Please add comments explaining that due to the FHIR spec is makes more sense for this to be a contained within the router
  2. Update persistence (bumping major version) - removing the method
  3. Update routing (bumping minor version - since the patch route would still be support but just in a different way)
  4. Update deployment package - (do we need to do anything to support a new header? application/json-patch+json)

For steps on setting up your local environment please check out our: https://github.com/awslabs/fhir-works-on-aws-deployment/blob/mainline/DEVELOPMENT.md

Since all of these PRs will be linked so heavily we will not merge the interface PR (and subsequently bump the npm version) until we are able to see the other changes code.

We look forward to this awesome contribution!

from fhir-works-on-aws-routing.

rsmayda avatar rsmayda commented on July 21, 2024

Thank you so much for a thoughtful response. Let us review this in depth, since like you mention it will be a backward breaking change. At face value this looks like a good logical approach but we will get back to you soon.

Just for our planning do you guys have a deadline?

from fhir-works-on-aws-routing.

lyndoh avatar lyndoh commented on July 21, 2024

Thanks, deadline is not too soon.

from fhir-works-on-aws-routing.

lyndoh avatar lyndoh commented on July 21, 2024

Thanks that all sounds good. We'll look to get started on this soon.

from fhir-works-on-aws-routing.

kcadette avatar kcadette commented on July 21, 2024

FHIR Works on AWS has been moved to maintenance mode. While in maintenance, we will not add any new features to this solution. All security issues should be reported directly to AWS Security at [email protected]. If you are new to this solution, we advise you to explore using HealthLake, which is our managed service for building FHIR based transactional and analytics applications. You can get started by contacting your AWS Account team. If you are an existing customer
of FHIR Works on AWS, and have additional questions or need immediate help, please reach out to [email protected] or contact your AWS Account team.

from fhir-works-on-aws-routing.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.