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

Task #446 #398: synchronize composer.json and ext_emconf #465

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kartolo
Copy link
Owner

@kartolo kartolo commented Dec 19, 2023

resolve #446 #398

@kartolo
Copy link
Owner Author

kartolo commented Dec 19, 2023

@baschny: please take a look. thanks.

@baschny
Copy link
Contributor

baschny commented Dec 20, 2023

That looks great, @kartolo thanks!

But if jumpurl is only "suggested", this should be noted in the documentation that this is required if you want to use the "click tracking" feature: https://docs.typo3.org/p/directmailteam/direct-mail/main/en-us/Configuration/EnablingClickStatistics/Index.html

In my opinion you could as well make it "required" and then the feature will work "out of the box" without needing to document anything further.

@kartolo kartolo changed the base branch from develop to master February 21, 2024 18:02
@kartolo
Copy link
Owner Author

kartolo commented Feb 21, 2024

Hint is added in documentation

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 this pull request may close these issues.

Make the TYPO3 dashboard extension optional
2 participants