edgedb-js
edgedb-js copied to clipboard
edgedb
Support changing Session within a Transaction
Add binding support for https://github.com/edgedb/edgedb/pull/7187
client..transaction((tx) => {
tx.withGlobals({ foo: 'bar' });
});
How should/can this be limited to only 4.8/5.0+ instances?
How should/can this be limited to only 4.8/5.0+ instances?
IMO, on earlier versions we continue to silently fail, which is exactly as bad as the current state.
IMO, on earlier versions we continue to silently fail, which is exactly as bad as the current state.
Well it was only a silent failure for me before because I was patching into the driver internals. Official types simply did not allow this.
Then I think it's fine to allow it in the types but have it not work on versions that do not support this. This is runtime-only information that I don't think makes sense to contort ourselves into making a compile-time error however nice that would be for consumers 😅