-
Notifications
You must be signed in to change notification settings - Fork 821
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
Customise React components through injection #6807
Comments
Unsetting estimate, that should be done in backlog grooming by the whole team. |
Trying to figure out how much language expressiveness we give up by going to a DI container. Same issue as |
@steve-silverstripe As a core contributor with a lot of React experience, I'd be keen to hear your thoughts on this as well. There's "customise frontend" related cards which are linked in the sections of the main RFC at #4887. This one is the basis for lots of this work: Customisation through dependency injection. Aaron is working on a PoC at the moment, which will go towards solidifying an API. |
I've written up some documentation. https://github.com/open-sausages/experimental-react-di |
As a SilverStripe developer, I want to be able to add, remove, or modify behaviour, state, or UI
in the React-powered areas of admin so that I can customise the application via a module or
my project.
Acceptance Criteria:
Tasks:
Injector.js
with new context-based solution.Apply BottleJS DI library to context-based solutionGalleryItem
TextField
behaviour. One to log character count, and one to check password strengthwindow.confirm
that if denied will kill its original actionRelated
Pull Requests
POCs
Documentation
#6957
The text was updated successfully, but these errors were encountered: