Skip to content
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_random_image] - no need field cache #23165

Merged
merged 1 commit into from Dec 1, 2018
Merged

Conversation

@alikon
Copy link
Contributor

@alikon alikon commented Nov 25, 2018

Pull Request for Issue #23162 .

Summary of Changes

removed field cache

Testing Instructions

module works as before

@ghost
Copy link

@ghost ghost commented Nov 25, 2018

I have tested this item successfully on 7cb1ee2<

@alikon Thanks for PR.


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

@Quy Quy commented Nov 25, 2018

I have tested this item successfully on 7cb1ee2


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

@Quy
Copy link
Contributor

@Quy Quy commented Nov 25, 2018

RTC


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

@mbabker mbabker added this to the Joomla 3.9.2 milestone Dec 1, 2018
@mbabker mbabker merged commit 91a1233 into joomla:staging Dec 1, 2018
3 of 4 checks passed
3 of 4 checks passed
continuous-integration/drone/pr the build failed
Details
@joomla-cms-bot
JTracker/HumanTestResults Human Test Results: 2 Successful 0 Failed.
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@alikon alikon deleted the alikon:patch-114 branch Dec 2, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants