Skip to content

Fixing the comment#43773

Closed
artur-stepien wants to merge 92 commits intojoomla:5.1-devfrom
artur-stepien:patch-2
Closed

Fixing the comment#43773
artur-stepien wants to merge 92 commits intojoomla:5.1-devfrom
artur-stepien:patch-2

Conversation

@artur-stepien
Copy link
Contributor

Fixing the comment in captcha object as it's clearly wrong.

Summary of Changes

Just fixing 2 comments to match what they describe.

Testing Instructions

Actual result BEFORE applying this Pull Request

Expected result AFTER applying this Pull Request

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

Hackwar and others added 30 commits March 7, 2024 10:33
Co-authored-by: Richard Fath <richard67@users.noreply.github.com>
[5.2] Change all calls for AbstractView::document getDocument()
[5.2] Adding missing type hints for $this
[5.2] Deprecate WebApplication::$JComponentTitle
pe7er and others added 12 commits June 18, 2024 10:10
* Show next run, show list by next run and show a warning when there are due tasks

* Update administrator/components/com_scheduler/src/Model/TasksModel.php

* Update administrator/components/com_scheduler/src/Model/TasksModel.php

* Update administrator/language/en-GB/com_scheduler.ini

Co-authored-by: Quy <quy@nomonkeybiz.com>

* fix default ordering

* Update administrator/language/en-GB/com_scheduler.ini

Co-authored-by: Quy <quy@nomonkeybiz.com>

* Dont fix things outside of the scope of this PR

---------
@chmst
Copy link
Contributor

chmst commented Jul 11, 2024

I have tested this item ✅ successfully on 84a4222

Code Review


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

@richard67
Copy link
Member

@artur-stepien The typos exist already in 4.4 so they should be fixed in the 4.4-dev branch. Release managers will later merge it up to the other branches when it has been merged into 4.4-dev.

Could you rebase to or redo on the 4.4-dev branch? Thanks in advance.

@richard67
Copy link
Member

@ChristineWk
Copy link

I have tested this item ✅ successfully on 84a4222

Code Review (J 4.4.6)


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

@artur-stepien artur-stepien changed the base branch from 5.2-dev to 4.4-dev August 6, 2024 09:20
@artur-stepien artur-stepien changed the base branch from 4.4-dev to 5.1-dev August 6, 2024 09:20
@artur-stepien artur-stepien deleted the patch-2 branch August 6, 2024 09:24
@richard67
Copy link
Member

@chmst @ChristineWk New PR for 4.4-dev is #43888 . Could you "test" that one, too? Thanks in advance.

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.

Comments