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

Invalid SPARQL query in SKOS client #24

Open
jaw111 opened this issue Jun 27, 2013 · 4 comments
Open

Invalid SPARQL query in SKOS client #24

jaw111 opened this issue Jun 27, 2013 · 4 comments

Comments

@jaw111
Copy link

jaw111 commented Jun 27, 2013

The query to create a new concept is not valid as it contains an empty WHERE {} clause.
Previously this was WHERE { ?s ?p ?o }.

Locally I modified the query to INSERT DATA in line the the query to create a top concept.

@tkurz
Copy link
Owner

tkurz commented Jun 27, 2013

Great stuff!

Can you send me a patch for this, would be nice to merge your local fixes back!

Best Regards
Thomas

@jaw111
Copy link
Author

jaw111 commented Jun 29, 2013

Hi Thomas,
What is the best method to do this, to be honest I'm still just getting started with Git and Github.
John

Date: Thu, 27 Jun 2013 05:33:43 -0700
From: [email protected]
To: [email protected]
CC: [email protected]
Subject: Re: [skosjs] Invalid SPARQL query in SKOS client (#24)

Great stuff!

Can you send me a patch for this, would be nice to merge your local fixes back!

Best Regards

Thomas


Reply to this email directly or view it on GitHub.

@tkurz
Copy link
Owner

tkurz commented Jun 30, 2013

Hi John

actually it is quite simple. Create the patch and append it to the issue. The command for it is git-format-patch

Here is a short description.
https://openhatch.org/wiki/How_to_generate_patches_with_git_format-patch

That is maybe the easiest way.

Thx in advance!
Thomas

Am 29.06.2013 um 11:36 schrieb jaw111 [email protected]:

Hi Thomas,
What is the best method to do this, to be honest I'm still just getting started with Git and Github.
John

Date: Thu, 27 Jun 2013 05:33:43 -0700
From: [email protected]
To: [email protected]
CC: [email protected]
Subject: Re: [skosjs] Invalid SPARQL query in SKOS client (#24)

Great stuff!

Can you send me a patch for this, would be nice to merge your local fixes back!

Best Regards

Thomas


Reply to this email directly or view it on GitHub.

Reply to this email directly or view it on GitHub.


Thomas Kurz
Knowledge and Media Technologies
Salzburg Research
Tel: +43/622/2288-253

@tkurz
Copy link
Owner

tkurz commented Oct 3, 2013

Hi John,

can you send me a pull request to your fork. This would be the easiest way to integrate your patch into the current master. It is described here: https://help.github.com/articles/using-pull-requests

Regards
Thomas

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

2 participants