User tests: Successful: Unsuccessful:
The problem was as @infograf768 said. We're using the $msg
way to show the message here.
Sample:
Labels |
Added:
?
|
Category | ⇒ | Administration UI/UX |
Rel_Number | ⇒ | 5786 | |
Relation Type | ⇒ | Pull Request for | |
Easy | No | ⇒ | Yes |
@Kubik-Rubik Hey. I've changed the fix to be in one line. Anything else?
@prasoon2211 Thank you, looks good for me. One more tester required!
Someone test this please?
On Jan 21, 2015 9:21 PM, "Viktor Vogel" notifications@github.com wrote:
@prasoon2211 https://github.com/prasoon2211 Thank you, looks good for
me. One more tester required!—
Reply to this email directly or view it on GitHub
#5809 (comment).
Status | Pending | ⇒ | Ready to Commit |
@test success moving to RTC Thanks @prasoon2211
Thank you for the PR @prasoon2211 and the test @zero-24!
Status | Ready to Commit | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-01-24 23:50:08 |
wow cool quick merge Thanks @Kubik-Rubik
GitHub notification mails make it possible! ;-)
Could you please optimize it to one line:
$app->enqueueMessage(JText::_('COM_INSTALLER_UNABLE_TO_FIND_INSTALL_PACKAGE'), 'error');
Thank you!