Skip to content

[4.0] Improve newsfeed list performance when filtering by tags#26495

Merged
wilsonge merged 2 commits intojoomla:4.0-devfrom
SharkyKZ:j4/query/newsfeeds/tag
Oct 8, 2019
Merged

[4.0] Improve newsfeed list performance when filtering by tags#26495
wilsonge merged 2 commits intojoomla:4.0-devfrom
SharkyKZ:j4/query/newsfeeds/tag

Conversation

@SharkyKZ
Copy link
Copy Markdown
Contributor

@SharkyKZ SharkyKZ commented Oct 6, 2019

Summary of Changes

This ports changes from #19284 to backend newsfeed list view.

Testing Instructions

Create some newsfeeds with tags.
View newsfeed list in backend.
Filter newsfeeds by single and multiple tags.

Expected result

Works like before.

Documentation Changes Required

No.

@richard67
Copy link
Copy Markdown
Member

I have tested this item ✅ successfully on 8964c14

About performance I can't say anything because my test data set is too small.

But I can confirm filtering by tags in the newsfeeds list works as well as before, carefully watching PHP error log with PHP error reporting set to maximum and watching database log.


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

@Quy
Copy link
Copy Markdown
Contributor

Quy commented Oct 7, 2019

I have tested this item ✅ successfully on 8964c14


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

@Quy Quy removed the PR-4.0-dev label Oct 7, 2019
@Quy
Copy link
Copy Markdown
Contributor

Quy commented Oct 7, 2019

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Oct 7, 2019
@wilsonge wilsonge merged commit 33669f3 into joomla:4.0-dev Oct 8, 2019
@wilsonge
Copy link
Copy Markdown
Contributor

wilsonge commented Oct 8, 2019

Thanks!

@joomla-cms-bot joomla-cms-bot added PR-4.0-dev and removed RTC This Pull Request is Ready To Commit labels Oct 8, 2019
@wilsonge wilsonge added this to the Joomla 4.0 milestone Oct 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants