Skip to content

Conversation

@JoshTheDerf
Copy link
Contributor

The usePermissions composable was completely overhauled for Directus 11, causing this extension to crash with an unexpected error on Directus 11 when used by non-Admin users.

This PR does a preliminary job of porting over the usePermissions changes to directus-extension-inline-form-interface. It hasn't been fully tested in all situations, but it does resolve the original crash when rendering the interface.

@hanneskuettner
Copy link
Owner

Thank you so much! This is a bit funny, since I'm the one of the people who implemented permissions in v11 and completely forgot about this little extension...

The only problem I see is that it copied code verbatim from the Directus repository which is licensed under BSL and can't be easily relicensed for this repo I'm afraid. I will think of a solution for that. Maybe this is the push we need to make the composables available as part of the extensions SDK 🤔

@JoshTheDerf
Copy link
Contributor Author

Hmm, that is quite a problem. I know a lot of extension developers do that, just because the internals aren't exposed, but I'm not exactly sure how to go about avoiding that for this extension. (As use-permissions.ts before was a similar situation.)

Might be something worth clarifying with the core team?

@davidpp
Copy link

davidpp commented Oct 16, 2024

Came across this PR while looking up the error after updating to Directus 11. Will fork your PR for now until the license issue is resolved. But wanted to leave a message to say thanks for both the extension from Hannes and for you taking time to fix the issue :)

@marcinkoziej
Copy link

Hi! Any updates on this? I am not sure if I should use the extension or look for alternatives (are there any?)

@DonCastillo
Copy link

Hi! Are there any updates on this? Thanks!

@xluckydegen
Copy link

Hi, is there any progress with this issue? It seems official implementation of this extension directly to Directus never happened, right? What is the best way how to fix it now? Do a fork and fix it by this? Thanks

@7oque
Copy link

7oque commented Nov 11, 2025

Heya, does anyone know what work needs to be done to this PR? I don't mind having a go. Also I assume there are some other extensions that needed similar changes and can be referenced? Or perhaps this feature is coming to directus natively soon? <3

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants