User tests: Successful: Unsuccessful:
Pull Request resolves #47489 .
Expired articles were displaying a green published icon instead
of the expired icon when selecting an article for a Single Article
menu item. Added a check for the publish_down date against the
current date before assigning the icon class, so expired articles
now correctly display the expired icon.
Expired articles show green published icon (icon-publish)
Expired articles show expired icon (icon-expired)
Please select:
Documentation link for guide.joomla.org:
[✅] No documentation changes for guide.joomla.org needed
Pull Request link for manual.joomla.org:
[✅] No documentation changes for manual.joomla.org needed
| Status | New | ⇒ | Pending |
| Category | ⇒ | Repository Administration com_admin SQL |
| Category | Repository Administration com_admin SQL | ⇒ | Administration com_content |
| Labels |
Added:
PR-5.4-dev
PR-6.1-dev
|
||
Thank you for the feedback @brianteeman!
I have updated the base branch to 6.1-dev and confirmed
the AI policy check. Sorry for the confusion!
the issue also happens on 5.4
as per https://github.com/joomla/joomla-cms?tab=readme-ov-file#which-branch-should-my-pull-request-target
Thank you for the feedback @brianteeman!
I have updated the base branch to 6.1-dev and confirmed the AI policy check. Sorry for the confusion!
@Chitrakshi180706 The change of the base branch was wrong. As @alikon has mentioned in his comment, this is a bug fix for a bug which happens also on 5.4.
So this PR should be made for the 5.4-dev branch.
Maybe a rebase would be not easy fro the upper to the lower version branch, and making a new PR for 5.4-dev would be easier.
Besides this, please check my 2 review suggestions above.
Thanks in advance.
Your branch is 6.1 but you are trying to merge it into 5.4