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

docs(cloudflare): cache population commands #101

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

james-elicx
Copy link
Collaborator

No description provided.

@james-elicx james-elicx marked this pull request as ready for review March 20, 2025 23:28
@james-elicx james-elicx requested a review from a team as a code owner March 20, 2025 23:28
@james-elicx james-elicx requested review from vicb and conico974 March 20, 2025 23:29
@james-elicx james-elicx force-pushed the james/populate-commands branch from af39b11 to f51790b Compare March 21, 2025 14:32
##### 2. Create tables for tag revalidations

The D1 tag cache requires two tables; one that keeps a record of the tag/path mappings, and another that tracks revalidation times.
The D1 database uses two tables; one that keeps a record of the tag/path mappings, and another that tracks revalidation times. These tables will be created when initialising the cache.
Copy link
Contributor

Choose a reason for hiding this comment

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

For all the D1 related change you could either rebase off or directly push some change to #99.
Or if you want me to integrate these change directly there, just let me know

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

i was going to do this pr based off of your one but didn't as the branch is in a fork. These changes are to the section not touched by your PR so there shouldn't be any conflicts

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.

2 participants