NPM Resource Changed ? Pending

User tests: Successful: Unsuccessful:

avatar wilsonge
wilsonge
12 May 2019

To test this ensure the privacy policy quickicon now correctly functions after the patch (before it always hangs on finding requests)

avatar wilsonge wilsonge - open - 12 May 2019
avatar wilsonge wilsonge - change - 12 May 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 12 May 2019
Category JavaScript Repository NPM Change Front End Plugins
avatar infograf768 infograf768 - change - 13 May 2019
Labels Added: NPM Resource Changed ?
avatar HLeithner HLeithner - change - 13 May 2019
Title
Port privacy check to es6 js and fix quickicon
[4.0] Port privacy check to es6 js and fix quickicon
avatar HLeithner HLeithner - edited - 13 May 2019
avatar infograf768
infograf768 - comment - 13 May 2019

I have tested this item successfully on c8557ba


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

avatar infograf768
infograf768 - comment - 13 May 2019

I have tested this item successfully on c8557ba


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

avatar infograf768 infograf768 - test_item - 13 May 2019 - Tested successfully
avatar wilsonge wilsonge - change - 13 May 2019
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-05-13 09:10:06
Closed_By wilsonge
avatar wilsonge wilsonge - close - 13 May 2019
avatar wilsonge wilsonge - merge - 13 May 2019
avatar wilsonge
wilsonge - comment - 13 May 2019

Thanks JM!

avatar brianteeman
brianteeman - comment - 13 May 2019

I have tested this item successfully on c8557ba


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

avatar brianteeman brianteeman - test_item - 13 May 2019 - Tested successfully
avatar infograf768
infograf768 - comment - 13 May 2019

Please though:
The button View Requests should align to right in LTR and left in RTL,
as it does for Extensions Update.
Text centered.

Screen Shot 2019-05-13 at 11 04 50

For extensions:

Screen Shot 2019-05-13 at 11 12 39

avatar infograf768
infograf768 - comment - 13 May 2019

Also it looks like we have a typo in the .js
langaugeStrings

avatar infograf768
infograf768 - comment - 13 May 2019

The wrong alignment of the warning comes from a missing
<div class="message-alert">
as we have for extension update

avatar wilsonge
wilsonge - comment - 13 May 2019

@infograf768 can you do a PR for that please?

avatar infograf768
infograf768 - comment - 13 May 2019

You mean towards yours or a new one?

avatar wilsonge
wilsonge - comment - 13 May 2019

New one. I already merged this before you posted

avatar infograf768
infograf768 - comment - 13 May 2019

argg ;)

avatar infograf768
infograf768 - comment - 13 May 2019

Please merge
#24882

Add a Comment

Login with GitHub to post a comment