-
Notifications
You must be signed in to change notification settings - Fork 7
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
Add schemaeditor support #11
Conversation
Looks good to me, though I never use any of the TTW functionality. Could you provide a test for this so we don't accidentally break this feature in the future? |
Cool. It will take a while for me to find time for those tests, but I'll write them :) |
3fbc542
to
28cd392
Compare
This seems to work and has to be merged! |
@Martronic-SA as mentioned above, tests are still missing for this. @datakurre any update on this? |
I'm sorry, I forgot. Within the next few days :) |
Excellent, thank you :) |
@lukasgraf Done. |
@datakurre awesome, thank you! 🎉 🍰 |
@Martronic-SA released |
Could someone familiar with schema editor framework to verify that this is the correct way to register a schema editor feature for behavior.
How this should work:
0. When behavior is not enabled, nothing happens