? ? ? NPM Resource Changed Pending

User tests: Successful: Unsuccessful:

avatar laoneo
laoneo
8 Nov 2021

Pull Request for discussion #35453 (comment).

Summary of Changes

Clears the search in media manager when folder is changed. Similar to a file browser on desktop computer.

@wilsonge and @bembelimen I leave that up to you guys if it should be added to 4.0 or 4.1.

Testing Instructions

  • Open the media manager
  • Search for banners
  • Double click on the banners folder
  • Search input field should be cleared

Actual result BEFORE applying this Pull Request

Search remains and banners folder is empty.

Expected result AFTER applying this Pull Request

Search input field is cleared and all items are shown.

avatar laoneo laoneo - open - 8 Nov 2021
avatar laoneo laoneo - change - 8 Nov 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 8 Nov 2021
Category JavaScript Administration com_media NPM Change Unit Tests
avatar laoneo laoneo - change - 8 Nov 2021
Labels Added: ? ? NPM Resource Changed
7dc55e1 11 Nov 2021 avatar laoneo cs
avatar ceford ceford - test_item - 24 Nov 2021 - Tested successfully
avatar ceford
ceford - comment - 24 Nov 2021

I have tested this item successfully on 7dc55e1

Works as described


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

avatar pritam825
pritam825 - comment - 2 Dec 2021

@ceford can you please test again, because when I applied patch it didn't getting me the output as expected, may be I am wrong, so can you please test once more, Thanks:)


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

avatar ceford
ceford - comment - 2 Dec 2021

For me it works exactly as described in the testing instructions. Did you run npm ci ?


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

avatar laoneo
laoneo - comment - 3 Dec 2021

@pritam825 did you run npm ci after you have applied the patch?

avatar jwaisner jwaisner - test_item - 13 Dec 2021 - Tested successfully
avatar jwaisner
jwaisner - comment - 13 Dec 2021

I have tested this item successfully on 7dc55e1


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

avatar jwaisner jwaisner - change - 13 Dec 2021
Status Pending Ready to Commit
avatar jwaisner
jwaisner - comment - 13 Dec 2021

RTC


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

avatar pritam825
pritam825 - comment - 15 Dec 2021

@laoneo sorry, I forgot to run npm ci , now it's fine

avatar pritam825 pritam825 - test_item - 15 Dec 2021 - Tested successfully
avatar pritam825
pritam825 - comment - 15 Dec 2021

I have tested this item successfully on 7dc55e1


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

avatar wilsonge wilsonge - close - 23 Dec 2021
avatar wilsonge wilsonge - merge - 23 Dec 2021
avatar wilsonge wilsonge - change - 23 Dec 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-12-23 23:40:56
Closed_By wilsonge
Labels Added: ?
avatar wilsonge
wilsonge - comment - 23 Dec 2021

Thanks!

Add a Comment

Login with GitHub to post a comment