Skip to content
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

TYPO3 13 Support #233

Closed
BokuNoMaxi opened this issue Oct 18, 2024 · 3 comments · Fixed by #260
Closed

TYPO3 13 Support #233

BokuNoMaxi opened this issue Oct 18, 2024 · 3 comments · Fixed by #260

Comments

@BokuNoMaxi
Copy link

As noted in Slack I started to upgrade your extension to be compatible with TYPO3 13 but I am kinda stuck in replacing the frontend Javascript, so that it works again.

The Finisher in the frontend works. so after submitting the form the record will be saved and is visible in the backend module. So only the frontend stuff is open to be implemented.

The export button doesn't work either, because of the error in the javascript, so I couldn't test the export, but I goes there it throws an error.

Link to the fork => https://github.com/BokuNoMaxi/typo3-formlog
what I did:

  • I let rector do the basic steps first and checked his changes, so there are a few code optimizations
  • I updated the backendmodule of the index Action so that it works as before
  • I moved the registration of the JavaScriptModules in the designated JavaScriptModules.php
  • Fixed the f:be.pageRender viewhelper
@bh-teufels
Copy link

@mbrodala have you been able to look at the fork of @BokuNoMaxi what is the status here?

@mbrodala
Copy link
Member

No, not yet. First a PR would be helpful (following the CGL) for reviewing.

@BokuNoMaxi
Copy link
Author

hadn't had the time to dig deeper into this topic, we talked about new JavaScript libraries on slack back then, but loading them the typo3 way was a bit too much for me. In my fork everything works except the backend module.

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 a pull request may close this issue.

3 participants