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

Updating core ontology prefixes #1

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

GavinMendelGleason
Copy link

This merely updates some of the core ontology prefixes to reflect our (soon to be documented) core ontologies.

This fix will make the tutorial run on TerminusDB v2.0.5 or greater but is not backward compatible with v2.0.4 and less. These versioning changes should have moved the major version number but unfortunately did not.

There are slight formatting changes which create a bit of a mixed message as most of the conditionals are to a house style, but I found it easier to read to understand what was going on.

This PR involves the following upgrades to be compliant with v3.0 of
TerminusDB (currently in dev branch, but soon to be in master).

* Changes that move master to main (to remark importance of BLM)

* Change that move api base to be at "/api". This simplifies console
development and third party applications on terminus servers.

* Movement of terminus_message and witness_message to a single
error_message which dispatches on the selection atom.

* Use of pcre patterns in error message matching to make matching more
flexible

* Some formatting changes of conditionals to reflect best practice,
includiong removal of "third branch false" which is implied.
@GavinMendelGleason
Copy link
Author

I've now three commits in this pull request. If accepted it will make the client current with v3.0 of TerminusDB which is not yet released but is currently in release candidate. This patch works in the dev branch of the TerminusDB-quickstart, and will work in master as soon as v3.0 is released.

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

Successfully merging this pull request may close these issues.

1 participant