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] Remove outdated comment #76

Closed
wants to merge 1 commit into from

Conversation

georgringer
Copy link

According to https://forge.typo3.org/issues/103142 this works

Copy link

@brotkrueml brotkrueml left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your PR.

The view helper documentation is rendered automatically from the docblocks of the class. Please change it there, then it should be changed also here after merging:
https://github.com/TYPO3/typo3/blob/main/typo3/sysext/fluid/Classes/ViewHelpers/Format/CaseViewHelper.php#L44

@andreaskienast
Copy link

Patch is pending at https://review.typo3.org/c/Packages/TYPO3.CMS/+/83034.

@georgringer
Copy link
Author

thanks!

@brotkrueml
Copy link

And already merged and rendered:
https://docs.typo3.org/other/typo3/view-helper-reference/12.4/en-us/typo3/fluid/latest/Format/Case.html

@georgringer Thanks for bringing this up.

@brotkrueml brotkrueml closed this Feb 20, 2024
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.

None yet

3 participants