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

[4.2] Deprecate a not used function in the modules HTML service #37638

Merged

Conversation

laoneo
Copy link
Member

@laoneo laoneo commented Apr 24, 2022

Summary of Changes

Since 5d2beea this function is not used in core anymore. This pr deprecates it so it can be later removed.

It was added in #29.

Can be merged on review.

@laoneo laoneo changed the title Deprecate a not used function in the modules HTML service [4.2] Deprecate a not used function in the modules HTML service Apr 24, 2022
@toivo
Copy link
Contributor

toivo commented Apr 30, 2022

I have tested this item ✅ successfully on e868c01


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/37638.

1 similar comment
@alikon
Copy link
Contributor

alikon commented Apr 30, 2022

I have tested this item ✅ successfully on e868c01


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/37638.

@alikon
Copy link
Contributor

alikon commented Apr 30, 2022

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/37638.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Apr 30, 2022
@roland-d roland-d merged commit 79bef58 into joomla:4.2-dev Apr 30, 2022
@roland-d roland-d deleted the j4/modules/deprecate-html-helper branch April 30, 2022 18:27
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Apr 30, 2022
@roland-d
Copy link
Contributor

Thank you.

@roland-d roland-d added this to the Joomla 4.2.0 milestone Apr 30, 2022
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.

6 participants