mike-jumper commented on PR #1111: URL: https://github.com/apache/guacamole-client/pull/1111#issuecomment-3254765921
> > LGTM ... @necouchman, are you OK with the scope of this being on `patch`, given that some documentation updates will be needed? Or should it be against `main`? > > It seems to me like it falls into the category of "New Feature" and should go to `main` instead of `patch`, and head for the 1.7.0 release? But there's some grey area, there, so I'm not going to be picky about that. Good with whatever you decide. Ah, OK - I'm happy to let this stay on `patch` given that: * The behavior of the vault extension is effectively broken for anyone using the "linked records" that this adds support for (confusing errors result otherwise). * The updates to the docs would be minimal (additional "CRITERIA" for those parameter tokens). -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
