? Pending

User tests: Successful: Unsuccessful:

avatar dgt41
dgt41
27 Mar 2017

Pull Request for Issue # .

Summary of Changes

  • Eliminate inline scripts
  • Drop some useless jQuery dependencies
  • Use .min.js (can someone do this as well in 3.x, this is not 2006)

Testing Instructions

Associations still work as before

Expected result

Actual result

Documentation Changes Required

avatar dgt41 dgt41 - open - 27 Mar 2017
avatar dgt41 dgt41 - change - 27 Mar 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 27 Mar 2017
Category Administration com_associations JavaScript
avatar dgt41 dgt41 - change - 27 Mar 2017
Labels Added: ?
avatar brianteeman
brianteeman - comment - 27 Mar 2017

cant this be done in 3.x

On 27 March 2017 at 21:51, Dimitri Grammatikogianni <
notifications@github.com> wrote:

Pull Request for Issue # .
Summary of Changes

  • Eliminate inline scripts
  • Drop some useless jQuery dependencies
  • Use .min.js (can someone do this as well in 3.x, this is not 2006)

Testing Instructions

Associations still work as before
Expected result Actual result Documentation Changes Required

You can view, comment on, or merge this pull request online at:

#14945
Commit Summary

  • com_associations

File Changes

Patch Links:


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#14945, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPH8aDGdRJ8xLUl1WdqlhRu9ySiLYxJks5rqCFkgaJpZM4MqzyM
.

--
Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
https://brian.teeman.net/ http://brian.teeman.net/

avatar dgt41
dgt41 - comment - 27 Mar 2017

@brianteeman it won't look good due to IE8

avatar infograf768
infograf768 - comment - 28 Mar 2017

The issue is that we can't test it as one can't install a language in 4.0

avatar infograf768
infograf768 - comment - 16 Apr 2017

Tried to test as we can now install language packs. Got a 500
#15329

avatar wilsonge
wilsonge - comment - 16 Apr 2017

@dgt41 you need to fix conflicts here please

6f3f1fb 17 Apr 2017 avatar dgt41 cs
avatar dgt41
dgt41 - comment - 17 Apr 2017

@wilsonge conflicts resolved!

avatar wilsonge wilsonge - change - 17 Apr 2017
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-04-17 13:03:20
Closed_By wilsonge
avatar wilsonge wilsonge - close - 17 Apr 2017
avatar wilsonge wilsonge - merge - 17 Apr 2017

Add a Comment

Login with GitHub to post a comment