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

Support for caching the changes in Code Files. #534

Closed
wants to merge 1 commit into from

Conversation

zymeli
Copy link

@zymeli zymeli commented Aug 27, 2023

Q A
Is bugfix?
New feature? ✔️
Breaks BC?
Fixed issues Support for caching the changes in Code Files.

@what-the-diff
Copy link

what-the-diff bot commented Aug 27, 2023

PR Summary

  • Enhancements to CodeFile.php
    The constructor has been modified to check if the current content matches with the original file content, setting the appropriate response. Additionally, new methods for setting and retrieving cache data have been incorporated, and there has been a revision in the 'save' and 'preview' methods for making use of the new caching system. Moreover, the 'diff' method now shows varying differences according to the cache data and actual file contents.

  • New Addition in DefaultController.php
    A method named 'actionCacheIt' has been added, enabling the setting of cache for specific files.

  • Updates to files.php view
    A new column titled "Cache It" has been introduced, providing a cache button for every file. Supporting JavaScript code has also been added to enable file caching when the button is clicked.

@samdark
Copy link
Member

samdark commented Oct 19, 2023

Thanks for contributing. What problem are you trying to solve with the pull request?

@samdark samdark added the pr:missing usecase It is not clear what is the use case for the pull request. label Oct 19, 2023
@samdark samdark closed this in c7f7533 Feb 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr:missing usecase It is not clear what is the use case for the pull request.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants