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

Rebuild mod_custom with new Joomla 4 structure #39931

Merged
merged 47 commits into from
Apr 17, 2023
Merged

Rebuild mod_custom with new Joomla 4 structure #39931

merged 47 commits into from
Apr 17, 2023

Conversation

sergeytolkachyov
Copy link
Contributor

Summary of Changes

Module Custom HTML has rebuilded with new Joomla 4 structure

Testing Instructions

Just try to use it as usual.

Actual result BEFORE applying this Pull Request

Module works

Expected result AFTER applying this Pull Request

Module works

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

@sergeytolkachyov
Copy link
Contributor Author

@richard67 what does the drone not like?

@alikon
Copy link
Contributor

alikon commented Feb 25, 2023

sergeytolkachyov and others added 2 commits February 25, 2023 18:01
Co-authored-by: Richard Fath <richard67@users.noreply.github.com>
@laoneo
Copy link
Member

laoneo commented Apr 17, 2023

As only code style issues are fixed since the tests and the new custom module system test is passing. I'm going to merge this one. @sergeytolkachyov thank you very much for your contribution. Just a small remark. In the future I would do an independent branch for your next pull request and not directly the 4.3-dev (or any other -dev) branch.

@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Apr 17, 2023
@laoneo laoneo added this to the Joomla! 4.4.0 milestone Apr 17, 2023
@sergeytolkachyov sergeytolkachyov deleted the 4.3-dev branch April 17, 2023 11:56
@sergeytolkachyov
Copy link
Contributor Author

As only code style issues are fixed since the tests and the new custom module system test is passing. I'm going to merge this one. @sergeytolkachyov thank you very much for your contribution. Just a small remark. In the future I would do an independent branch for your next pull request and not directly the 4.3-dev (or any other -dev) branch.

Hello. Thanks for remark. So, how will I do: create fork, then create branch?

@laoneo
Copy link
Member

laoneo commented Apr 17, 2023

Yes, create a branch from the 4.4 branch, which must be identically to the one from this repository.

richard67 added a commit to richard67/joomla-cms that referenced this pull request May 1, 2023
richard67 added a commit to richard67/joomla-cms that referenced this pull request May 1, 2023
richard67 added a commit to richard67/joomla-cms that referenced this pull request May 9, 2023
richard67 added a commit to richard67/joomla-cms that referenced this pull request May 22, 2023
richard67 added a commit to richard67/joomla-cms that referenced this pull request May 23, 2023
laoneo pushed a commit that referenced this pull request May 23, 2023
* Add deleted files from #40066 , #40092 , #40142 and #40146

* Add deleted files from #40180 and #40174

* Add deleted files from #39527 , #39535 , #40169 and #40173

* Add deleted files from #39552 , #39931 , #39628 and #40328

* Add deleted files from #40216

* Add deleted files from #40561 , #40190 and #40205
richard67 added a commit to richard67/joomla-cms that referenced this pull request May 27, 2023
richard67 added a commit to richard67/joomla-cms that referenced this pull request Jun 3, 2023
richard67 added a commit to richard67/joomla-cms that referenced this pull request Jun 8, 2023
richard67 added a commit to richard67/joomla-cms that referenced this pull request Jun 23, 2023
richard67 added a commit to richard67/joomla-cms that referenced this pull request Jul 3, 2023
richard67 added a commit to richard67/joomla-cms that referenced this pull request Jul 23, 2023
richard67 added a commit to richard67/joomla-cms that referenced this pull request Aug 12, 2023
richard67 added a commit to richard67/joomla-cms that referenced this pull request Aug 19, 2023
richard67 added a commit to richard67/joomla-cms that referenced this pull request Aug 20, 2023
richard67 added a commit to richard67/joomla-cms that referenced this pull request Aug 22, 2023
richard67 added a commit to richard67/joomla-cms that referenced this pull request Sep 1, 2023
richard67 added a commit to richard67/joomla-cms that referenced this pull request Sep 4, 2023
richard67 added a commit to richard67/joomla-cms that referenced this pull request Sep 4, 2023
richard67 added a commit to richard67/joomla-cms that referenced this pull request Sep 14, 2023
richard67 added a commit to richard67/joomla-cms that referenced this pull request Sep 19, 2023
richard67 added a commit to richard67/joomla-cms that referenced this pull request Sep 26, 2023
richard67 added a commit to richard67/joomla-cms that referenced this pull request Oct 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.