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

new links are "merged" into existing documents #31

Open
abalmos opened this issue Jan 22, 2020 · 3 comments
Open

new links are "merged" into existing documents #31

abalmos opened this issue Jan 22, 2020 · 3 comments

Comments

@abalmos
Copy link
Member

abalmos commented Jan 22, 2020

For example, if at

/bookmarks/services/trellis/signer there is an object

{
   "jobs": {}
}

and then one PUTs

{
  "_id": "resources/123"
}

to /bookmarks/services/trellis/signer.

In this case, /bookmarks/services/trellis/signer/blah correctly follows the new link. However getting /bookmarks/services/trellis gives back something like:

{
  "jobs": {},
  "_id": "resources/123"
}

and jobs is not ever accessible.

It appears PUTs of links are "merged" into existing paths but that doesn't make a lot of sense.

@awlayton
Copy link
Member

I suppose OADA treats links as a primitive of sorts, so in that case one might expect the link to replace whatever was there?

@aultac
Copy link
Member

aultac commented Mar 13, 2021 via email

@awlayton
Copy link
Member

It would make the query more complicated yes. We would need to do something with MERGE_RECURSIVE rather than simply doing an UPSERT with the changes as we do now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants