Feeling Lucky
?
[#8173] - Redo of #4563
- Closed
- 27 Oct 2015
- Medium
- Build: staging
- Easy Test
- # 8173
- Diff
- dgt41:____redoMessagesModal
- Success The Travis CI build passed Details
User tests: Successful: Unsuccessful:
The PR #4563 was broken into two parts
in order to get it merged in the messages repo. But that didn’t happen so I just did that missing part here.
In short this PR drops the mootools modal for the config modal of the component messages.
To test simply check that the modal still operates and the configuration gets saved in com_messages
| Status | New | ⇒ | Pending |
| Labels |
Added:
?
|
||
zero-24
- comment
- 27 Oct 2015
I have tested this item 
successfully on 431b8ea
Works great. Sorry for taking it so long :( And thank you for doing it 
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8173.
| Category | ⇒ | Administration UI/UX |
| Milestone |
Added: |
||
| Easy | No | ⇒ | Yes |
zero-24
- comment
- 27 Oct 2015
Thanks @bertmert if @dgt41 gets the time to fix Travis (merging my PR or do it with a similiar commit we can RTC it for 3.4.6
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8173.
| Milestone |
Added: |
||
joomla-cms-bot
- comment
- 27 Oct 2015
This PR has received new commits.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8173.
| Status | Pending | ⇒ | Ready to Commit |
zero-24
- comment
- 27 Oct 2015
RTC 
Thanks.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8173.
| Labels |
Added:
?
|
||
rdeutz
- | Status | Ready to Commit | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-10-27 21:01:35 |
| Closed_By | ⇒ | rdeutz |
| Labels |
Removed:
?
|
||
| Milestone | |||
| Milestone |
Added: |
||
| Milestone |
Added: |
||
| Milestone | |||
Thanks @dgt41 see: dgt41#24 for Travis. ;)