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

Remove (or restore) user-defined schematron feature #285

Closed
lmsurpre opened this issue Oct 17, 2019 · 2 comments
Closed

Remove (or restore) user-defined schematron feature #285

lmsurpre opened this issue Oct 17, 2019 · 2 comments
Assignees
@lmsurpre
Milestone
Sprint 3

Comments

@lmsurpre
Copy link
Member

In DSTU2, we supported a way for admins to configure the server with "user-defined schematron" that would fire under certain conditions (e.g. whenever we get an Observation resource).

In R4, we're trying to do away with the reliance on XML serialization for validation (primarilly for performance reasons) and I think we already removed the XSLT libraries needed for this feature.

I think we should "finish the job" and totally remove all traces of this feature before we have our initial R4 release.
@lmsurpre lmsurpre added this to the Sprint 3 milestone Oct 17, 2019
@lmsurpre
Copy link
Member Author

lmsurpre commented Oct 17, 2019 •
edited
Loading

similar to #153, but for a different feature

@lmsurpre lmsurpre self-assigned this Oct 18, 2019
lmsurpre added a commit that referenced this issue Oct 25, 2019
@lmsurpre
issue #285 - remove schematron config from all config files
c809ddb 
Signed-off-by: Lee Surprenant <lmsurpre@us.ibm.com>
lmsurpre added a commit that referenced this issue Oct 25, 2019
@lmsurpre
issue #285 - remove schematron property references from
408754a 
also removed references to virtual resources as able.  needs more work but thats issue #2.

Signed-off-by: Lee Surprenant <lmsurpre@us.ibm.com>
lmsurpre added a commit that referenced this issue Oct 25, 2019
@lmsurpre
issue #285 - remove schematron property references from
2542fa9 
also removed references to virtual resources as able.  needs more work but thats issue #2.

Signed-off-by: Lee Surprenant <lmsurpre@us.ibm.com>
@prb112
Copy link
Contributor

prb112 commented Oct 28, 2019

worked for me

@prb112 prb112 closed this as completed Oct 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lmsurpre lmsurpre

Labels
None yet
Projects
None yet
Milestone
Sprint 3
Development

No branches or pull requests
2 participants
@prb112 @lmsurpre