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

Add persisted entity id to JSON-LD payload as @id #15

Closed
sohkai opened this issue Oct 5, 2016 · 2 comments
Closed

Add persisted entity id to JSON-LD payload as @id #15

sohkai opened this issue Oct 5, 2016 · 2 comments
Assignees

Comments

@sohkai
Copy link
Contributor

sohkai commented Oct 5, 2016

We currently ignore the persisted ids of returned entities, which makes it very difficult for users to save the results on their own backends as well as use other parts of the interface that require an entity's id (e.g. transferring rights).

Although not technically correct, as we don't support GET operations yet (see #14), we can use the @id in the returned JSON-LD to denote the persisted id. Doing so results in a relative URL to the current route, which works for /manifestations and /rights, but will require some special handling for RightsAssignments since its route is /rights/transfer (e.g. its @id will probably look something like ../rights_assignment/<id>).

@sohkai sohkai self-assigned this Oct 5, 2016
This was referenced Oct 5, 2016
@TimDaub
Copy link
Contributor

TimDaub commented Oct 12, 2016

@sohkai If this is blocking you might wanna add this to: bigchaindb/org#22

@sohkai
Copy link
Contributor Author

sohkai commented Oct 12, 2016

@TimDaub actually the PR's available at #16. Transfers are blocked by this, so I treated it as a subtask of its implementation.

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