New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[mod_syndicate] - no need field cache #23166

Merged
merged 1 commit into from Dec 1, 2018

Conversation

Projects
None yet
6 participants
@alikon
Copy link
Contributor

alikon commented Nov 25, 2018

Pull Request for Issue #23163 .

Summary of Changes

removed field cache

Testing Instructions

module works as before

@franz-wohlkoenig

This comment has been minimized.

Copy link

franz-wohlkoenig commented Nov 25, 2018

I have tested this item successfully on 07c312e

@alikon Thanks for this PR too.


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

This comment has been minimized.

Copy link
Contributor

Quy commented Nov 25, 2018

I have tested this item successfully on 07c312e


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

@Quy

This comment has been minimized.

Copy link
Contributor

Quy commented Nov 25, 2018

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC label Nov 25, 2018

@infograf768

This comment has been minimized.

Copy link
Member

infograf768 commented Nov 26, 2018

@alikon
We need similar PR for mod_languages

@alikon

This comment has been minimized.

Copy link
Contributor Author

alikon commented Nov 26, 2018

@infograf768 when you can please test #23174

@mbabker mbabker added this to the Joomla 3.9.2 milestone Dec 1, 2018

@mbabker mbabker merged commit 2657382 into joomla:staging Dec 1, 2018

4 checks passed

JTracker/HumanTestResults Human Test Results: 2 Successful 0 Failed.
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/drone/pr the build was successful
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@joomla-cms-bot joomla-cms-bot added PR-staging and removed RTC labels Dec 1, 2018

@alikon alikon deleted the alikon:patch-115 branch Dec 2, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment