Skip to content

Conversation

RJKeevil
Copy link

Description

Demonstrates the issue raised in #265

@RJKeevil RJKeevil changed the title WIP: Add example from JSON Data Format docs to v2 tests Add example from JSON Data Format docs to v2 tests Sep 1, 2025
@RJKeevil RJKeevil marked this pull request as ready for review September 1, 2025 09:21
@RJKeevil RJKeevil requested a review from a team as a code owner September 1, 2025 09:21
Note this currently fails intentionally, to demonstrate mismatch with documentation
@CLAassistant
Copy link

CLAassistant commented Oct 1, 2025

CLA assistant check
All committers have signed the CLA.

@RJKeevil
Copy link
Author

RJKeevil commented Oct 1, 2025

@matthewmcneely Updated this PR now v2 is merged with v1. I think its very important to get this fixed as 1) the DQL docs are incompatatible with RunDQL, and 2) it's the single reason that I cannot use the new structure without rewriting hundreds of mutations.

@matthewmcneely
Copy link
Member

it's the single reason that I cannot use the new structure without rewriting hundreds of mutations

@RJKeevil You should have a closer look at the final APIs for v25. The namespace concept was scaled back. You can create namespaces via gRPC, but you cannot name them (they just allocate the next ns ID and return an integer)... and you must assign users in those namespaces and login the old way for operations to take place in their respective namespaces.

In this light, you might as well keep your old transaction/setJSON code.

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

Successfully merging this pull request may close these issues.

3 participants