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

kirby 3 rewrite #4

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

kirby 3 rewrite #4

wants to merge 4 commits into from

Conversation

bnomei
Copy link

@bnomei bnomei commented Jul 23, 2018

but did not do component to cache pages.
docs/kirby-3-version.md show basic test.

Signed-off-by: Bruno Meilick b@bnomei.com

but did not do component to cache pages

Signed-off-by: Bruno Meilick <b@bnomei.com>
@bnomei
Copy link
Author

bnomei commented Jul 23, 2018

oh yes. i do not actually want to commit to master but there is not kirby-3 branch yet.

Signed-off-by: Bruno Meilick <b@bnomei.com>
Signed-off-by: Bruno Meilick <b@bnomei.com>
@bnomei
Copy link
Author

bnomei commented Jul 24, 2018

rewrote code to new cache which makes this plugin kind of useless but might be nice if porting k2 pages to k3 – kind of a wrapper.

https://www.notion.so/Cache-Concept-3b7ba5af6ddf4863b1344fc7b433a729

@bnomei
Copy link
Author

bnomei commented Aug 31, 2018

did my own version with minor changes i case you are interested:
https://github.com/bnomei/kirby3-lapse

@jenstornell
Copy link
Collaborator

jenstornell commented Aug 31, 2018

@bnomei If it's good enough (which means works great and is simple enough), you can take over this feature with your plugin. When Kirby 3 is released I'll close mine.

@bnomei
Copy link
Author

bnomei commented Aug 31, 2018

its dead simple like yours before. i just added the option to force updating the cache, which i needed once or twice before, and moved from seconds to minutes (like kirby cache does now)

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.

None yet

2 participants