? Success

User tests: Successful: Unsuccessful:

avatar andrepereiradasilva
andrepereiradasilva
29 Jun 2016

Summary of Changes

Very simple PR: the send test mail message type should be message, not success.
There is no success message type.

Types of messages:

  • message: green
  • warning: yellow
  • notice: blue
  • error: red

Note that all other send test mail messages (error warnings) have the title.

Testing Instructions

Code review.

But if you really want to test:

  • Before patch "Send Test mail" message when a link is sent the resulting js green message doesn't have title.
  • After patch there is a "Message" title in the green message (like already works in case of warnings or errors).
avatar andrepereiradasilva andrepereiradasilva - open - 29 Jun 2016
avatar andrepereiradasilva andrepereiradasilva - change - 29 Jun 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 29 Jun 2016
Labels Added: ?
avatar andrepereiradasilva andrepereiradasilva - change - 29 Jun 2016
The description was changed
avatar andrepereiradasilva andrepereiradasilva - change - 29 Jun 2016
The description was changed
avatar andrepereiradasilva andrepereiradasilva - change - 29 Jun 2016
The description was changed
avatar andrepereiradasilva andrepereiradasilva - change - 29 Jun 2016
The description was changed
avatar ggppdk ggppdk - test_item - 29 Jun 2016 - Tested successfully
avatar ggppdk
ggppdk - comment - 29 Jun 2016

I have tested this item successfully on ebba6b8

As described, after patch the message text and color are the same, but also there is a title "Message"


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

avatar andrepereiradasilva andrepereiradasilva - change - 29 Jun 2016
The description was changed
avatar wojsmol wojsmol - test_item - 29 Jun 2016 - Tested successfully
avatar wojsmol
wojsmol - comment - 29 Jun 2016

I have tested this item successfully on ebba6b8

code review


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

avatar brianteeman brianteeman - change - 29 Jun 2016
Category Code style
avatar brianteeman brianteeman - change - 29 Jun 2016
Status Pending Ready to Commit
avatar brianteeman
brianteeman - comment - 29 Jun 2016

Rtc


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

avatar joomla-cms-bot joomla-cms-bot - change - 29 Jun 2016
Labels Added: ?
avatar BurtNL BurtNL - test_item - 29 Jun 2016 - Tested successfully
avatar BurtNL
BurtNL - comment - 29 Jun 2016

I have tested this item successfully on ebba6b8


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

avatar brianteeman brianteeman - change - 8 Jul 2016
Milestone Added:
avatar roland-d roland-d - change - 16 Jul 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-07-16 08:33:11
Closed_By roland-d
avatar roland-d roland-d - close - 16 Jul 2016
avatar roland-d roland-d - merge - 16 Jul 2016
avatar joomla-cms-bot joomla-cms-bot - close - 16 Jul 2016
avatar joomla-cms-bot joomla-cms-bot - change - 16 Jul 2016
Labels Removed: ?

Add a Comment

Login with GitHub to post a comment