NPM Resource Changed ? ? Pending

User tests: Successful: Unsuccessful:

avatar hans2103
hans2103
5 Jul 2020

Pull Request for Issue # .

Summary of Changes

continue from #29984 (comment)
Searched for dragable and replaced by draggable

avatar hans2103 hans2103 - open - 5 Jul 2020
avatar hans2103 hans2103 - change - 5 Jul 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 5 Jul 2020
Category Administration com_banners com_categories com_contact com_content com_menus com_modules com_newsfeeds com_plugins com_tags com_workflow JavaScript Repository NPM Change
avatar ceford
ceford - comment - 6 Jul 2020

I found 29 instances before patching and 18 after - all in media / system / js which I did not rebuild.


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

avatar hans2103 hans2103 - change - 6 Jul 2020
Labels Added: NPM Resource Changed ?
avatar hans2103
hans2103 - comment - 6 Jul 2020

@ceford That is due to the fact that all files in /media are in .gitignore
I cannot create a pull request for that one.


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

avatar zero-24
zero-24 - comment - 6 Jul 2020

The media folder should not have that any more after running npm. An other way to test that would be the pre build packages there npm should have rebuild them already.

avatar hans2103
hans2103 - comment - 6 Jul 2020

@zero-24 is there anything I have to change on my PR?

avatar zero-24
zero-24 - comment - 6 Jul 2020

@zero-24 is there anything I have to change on my PR?

No all is fine :)

avatar bonzani bonzani - test_item - 13 Jul 2020 - Tested successfully
avatar bonzani
bonzani - comment - 13 Jul 2020

I have tested this item successfully on 1923fb5


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

avatar Quy Quy - test_item - 14 Jul 2020 - Tested successfully
avatar Quy
Quy - comment - 14 Jul 2020

I have tested this item successfully on 1923fb5


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

avatar Quy Quy - change - 14 Jul 2020
Status Pending Ready to Commit
avatar Quy
Quy - comment - 14 Jul 2020

RTC


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

avatar Quy
Quy - comment - 14 Jul 2020

RTC


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

avatar Quy
Quy - comment - 15 Jul 2020

Can you please update to include changes in #30094?

avatar hans2103
hans2103 - comment - 15 Jul 2020

@Quy merged changes and found two new files with dragable instead of draggable.

avatar hans2103
hans2103 - comment - 15 Jul 2020

I have added the merge in commit hans2103@89c852e
I added the changes of the two files in commit hans2103@e7490ea

They are in branch https://github.com/hans2103/joomla-cms/commits/feature/draggable-typo-fixes
But I don't see them in this GitHub issue.
Any idea why?

avatar hans2103 hans2103 - change - 15 Jul 2020
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - change - 15 Jul 2020
Category Administration com_banners com_categories com_contact com_content com_menus com_modules com_newsfeeds com_plugins com_tags com_workflow JavaScript Repository NPM Change Administration com_banners com_categories com_contact com_content com_fields com_menus com_modules com_newsfeeds com_plugins com_tags com_workflow JavaScript Repository NPM Change
avatar Quy
Quy - comment - 15 Jul 2020

They have been added to your PR here. See e7490ea

Thank you!!!

avatar hans2103
hans2103 - comment - 15 Jul 2020

@Quy yes! Now I see them too.

avatar richard67 richard67 - change - 20 Jul 2020
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-07-20 19:06:11
Closed_By richard67
avatar richard67 richard67 - close - 20 Jul 2020
avatar richard67 richard67 - merge - 20 Jul 2020
avatar richard67
richard67 - comment - 20 Jul 2020

Thanks!

Add a Comment

Login with GitHub to post a comment