-
Notifications
You must be signed in to change notification settings - Fork 5
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
v11, long term. Compatibility with future versions #5
Comments
I havent personally tested the extension with v11 yet however it should work 🤔 None of the functionalities this extension relies on have been changed since 10.10 afaik |
@br41nslug yeah I see, what do you think, are there any chances similar functionality will land in directus sometime in the future? |
I somewhat doubt it considering the "best" (most performant) way to implement this is on the SQL level like (in postgres):
But I am not sure this is consistently supported for all database providers supported by Directus. |
@br41nslug does directus support running queries like that natively? |
You can from within backend extensions like this, but when using raw SQL you'll have to be sure of the underlying database compatibility if you'd want to distribute that extension |
just dig a bit deeper directus/directus#12452 tho this is the best pointer I've found so far directus/directus#3312 |
Thank you for great extension ⭐
This is more a question rather than an issue.
As it seems, this extension was created as a follow-up to this discussion. The current major version of Directus is 11 tho as per documentation
tested with Directus 10.10.4.
I wonder if this extension has been battle-tested with v11 and if it is safe to rely on that going forward, as it seems there is still no native functionality like that.
The text was updated successfully, but these errors were encountered: