? Success

User tests: Successful: Unsuccessful:

avatar dgt41
dgt41
8 Nov 2014

Executive summary

This PR converts the form validation on com_messages to use plain jquery (no mootools call on every form

avatar dgt41 dgt41 - open - 8 Nov 2014
avatar jissues-bot jissues-bot - change - 8 Nov 2014
The description was changed
Labels Added: ?
avatar brianteeman brianteeman - change - 8 Nov 2014
Category JavaScript
avatar smanzi
smanzi - comment - 11 Nov 2014

@dgt41 Unable to select recipient (users list not populated), but I have the same behavior without your PR! New issue??

avatar dgt41
dgt41 - comment - 11 Nov 2014

@smanzi You just found a new bug!

avatar smanzi
smanzi - comment - 11 Nov 2014

@dgt41 do you want open new issue and try a PR? I'm currently busy fixing up JHtmlBoostrap::renderModal() (yes, it has issues!), but after that I can give a look too...

avatar dgt41
dgt41 - comment - 11 Nov 2014

@smanzi I am kinda busy right now, maybe later

avatar smanzi
smanzi - comment - 13 Nov 2014

@test success

@dgt41 there was no bug here! It was only due to the fact that I (and probably you too...) didn't have any other user beside ourselves enabled to send/receive messages. I think it must be an Administrator or Super User. Try adding a new Administrator and it will work. You should probably add this to your test instructions...

The question now is: if the recipient must be an Administrator/Super User, why in the modal for user selection it is possible to select also the other user groups for filtering? :smirk:

avatar anibalsanchez
anibalsanchez - comment - 13 Nov 2014

@test success

I agree, UI is inconsistent. I think messages component is not used and it can be removed (similar case to weblinks).

avatar smanzi
smanzi - comment - 13 Nov 2014

... and Banners and Contacts and (IMHO) Newsfeeds

avatar dgt41
dgt41 - comment - 13 Nov 2014

@anibalsanchez @smanzi my list of useless stuff is a little bit longer

avatar smanzi
smanzi - comment - 13 Nov 2014

@dgt41 @anibalsanchez If we start looking into plugins it can be way longer!!

avatar anibalsanchez
anibalsanchez - comment - 13 Nov 2014

Yeah! Let's kick them out.

avatar dgt41
dgt41 - comment - 13 Nov 2014

@anibalsanchez @smanzi According to the roadmap in two days time will have 3.5 without com_newsfeeds, com_banners, com_contacts, com_search, com_finder, com_redirect, com_messages. But then again 3.4 is not yet released...

avatar anibalsanchez
anibalsanchez - comment - 13 Nov 2014

Well, let's wait for the next version.

avatar Bakual
Bakual - comment - 13 Nov 2014

The roadmap is flexible :smile:
If it's ready before the release of 3.4 beta, it can be considered for 3.4. If not, it will go into 3.5.

avatar dgt41
dgt41 - comment - 13 Nov 2014

@Bakual Thomas I would like to help there, but I guess someone has to make all these repos...

avatar Bakual
Bakual - comment - 13 Nov 2014

I think ss soon as the 3.4 is released, we will create all those repos.

avatar dgt41
dgt41 - comment - 26 Nov 2014

@roland-d this one got 2 good test :)

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

avatar smanzi smanzi - test_item - 26 Nov 2014 - Tested successfully
avatar roland-d roland-d - alter_testresult - 26 Nov 2014 - anibalsanchez: Tested successfully
avatar roland-d roland-d - change - 26 Nov 2014
Status Pending Ready to Commit
avatar roland-d
roland-d - comment - 26 Nov 2014

Moving to RTC, 2 successful tests.

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

avatar brianteeman brianteeman - change - 29 Nov 2014
Labels Added: ?
avatar smanzi
smanzi - comment - 30 Nov 2014

@test success for this PR
Tested the set of: #5046 #5048 #5049 #5050 #5051 #5052 #5053 #5054 #5056 #5058 #5113 #5255

avatar phproberto phproberto - close - 30 Nov 2014
avatar zero-24 zero-24 - close - 30 Nov 2014
avatar phproberto phproberto - change - 30 Nov 2014
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2014-11-30 01:58:44
avatar zero-24 zero-24 - change - 14 Oct 2015
Labels Removed: ?

Add a Comment

Login with GitHub to post a comment