Awesome work @tbrisker! thanks for doing it
opened also feat(vendor-core): add @reduxjs/toolkit library by laviro · Pull Request #319 · theforeman/foreman-js · GitHub
This RFC was discussed again today on the UX Interest group meeting and the remaining concerns were addressed. Unless anyone raises any further concerns I believe we can consider this accepted. Next steps would be:
merge @Ron_Lavi’s PR adding the redux toolkit library in to foreman-js
releasing a new version of @theforeman/vendor
- 8.8.0 released
update packaging to use newer @theforeman/vendor
- Bump foreman-js by tbrisker · Pull Request #6873 · theforeman/foreman-packaging · GitHub
update docs to recommend the new structure
profit!
Who is taking care of which task?
I can write the docs
The updated vendor package has been released and I’ve turned my POC PR into a real one that also requires the newer vendor version and implements the new folder structure.