Skip to content
This repository has been archived by the owner on Oct 12, 2021. It is now read-only.

Include JSON-LD MIME in isWriteable check #45

Merged
merged 2 commits into from Nov 17, 2018
Merged

Include JSON-LD MIME in isWriteable check #45

merged 2 commits into from Nov 17, 2018

Conversation

berezovskyi
Copy link
Member

fixes #44

@berezovskyi
Include JSON-LD MIME in isWriteable check; fixes #44
afac23c 
Signed-off-by: Andrew Berezovskyi <andriib@kth.se>
@berezovskyi berezovskyi self-assigned this Nov 16, 2018
@berezovskyi berezovskyi mentioned this pull request Nov 16, 2018
Closed
@jadelkhoury
Copy link
Contributor

jadelkhoury commented Nov 17, 2018 •
edited

+1 verifed.
@berezovskyi we needed to add the jsonld support for query results. I fixed that with the second commit.
1 question remains: shouldn't we change the pom-files in this patch to be "snapshot" before we merge?

@berezovskyi
Copy link
Member Author

Already done ;) https://github.com/eclipse/lyo.core/blob/master/oslc4j-core-build/pom.xml#L6

@jadelkhoury
Copy link
Contributor

Merge?

@berezovskyi berezovskyi merged commit d1f2afb into master Nov 17, 2018
@berezovskyi berezovskyi deleted the b44-jsonld-not-writeable branch November 17, 2018 19:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@jadelkhoury jadelkhoury Awaiting requested review from jadelkhoury

Assignees

@berezovskyi berezovskyi

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

OslcJsonLdProvider never called when trying to write to 'application/ld+json'
2 participants
@berezovskyi @jadelkhoury