Skip to content

Conversation

@pxpm
Copy link
Contributor

@pxpm pxpm commented Jun 6, 2025

This creates a general CacheSetup class, that can be used by any entities that need to store some kind of setupClosure in the cache.

At the moment I've created the DatatableCache that stores the setup of the datatable in the cache.

pxpm and others added 2 commits June 6, 2025 15:08
Copy link
Member

@tabacitu tabacitu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Niiiice! Something irks my spidey sense a bit here, so I want to take another look at this on Monday with a fresh brain.

Copy link
Member

@tabacitu tabacitu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We. Are. DONE!!

RUN WITH IT, PEDRO! Merge it!

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gold. We're done here. In this file I mean.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is very clean. And well-named. I like it. We're done here.

@tabacitu tabacitu merged commit a974b45 into datatable-single-component Jun 9, 2025
27 checks passed
@tabacitu tabacitu deleted the create-setup-cache-class branch June 9, 2025 13:03
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.

4 participants