-
Notifications
You must be signed in to change notification settings - Fork 9
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
[FEATURE] Update extension for TYPO3 v12 support #35
Conversation
d72375a
to
dca1f9d
Compare
The update adds a new controller, DataSheetImportController, for handling data import from excel sheets. This includes supporting files, templates and necessary configurations. Also, there are enhancements to user access check functions with better considerations for user permissions. A testing framework is also being introduced though actual tests are not yet added. This is preparation work for updating this extension for v12/v13 support
dca1f9d
to
90853a9
Compare
9d16021
to
3757aee
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TBH only did a pure code review only, no testing in running instance for any versions.
Nice move, but some minor remarks from my side. 😇
} | ||
|
||
/** @deprecated Function call will be removed in 6.0 */ | ||
$insertData = $this->callHook($insertData, $importJob->getTable()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See comment in the callHook()
method. I would suggest to consider dropping the hook directly.
Successfully tested the following functions in a running v12.4.9 instance:
|
63c1718
to
7a68480
Compare
With this PR the extension is shippable for TYPO3 v12. This adds support for the backend integration.
7a68480
to
9674576
Compare
With this PR the extension is shippable for TYPO3 v12. This adds support
for the backend integration.
This PR is ready for testing