@andypost at #2603046-4: Support anonymous users:

Also a line in README would be helpful about how to use the module with 2 core's caching modules

Comments

Wim Leers created an issue. See original summary.

Wim Leers’s picture

Status: Active » Needs review

There's a soft dependency of BigPipe on Dynamic Page Cache. But, 99% of the time, you'll actually want Dynamic Page Cache to be enabled. So I think I should make that a requirement. Thoughts, @Fabianx?

However, there's no relation at all to the Page Cache module, except that Page Cache makes anon as fast as possible, BigPipe + Dynamic Page Cache makes auth as fast as possible. Perhaps that's what you're alluding to, @andypost?

andypost’s picture

I'm +1 on hard dependency and warning at status page that you need page cache module enabled - like a cal to saninty
Also would be great to update project page with a simple steps:
1) enable module and check status page
2) docs page about tuning the cache is...
3) docs page about module usage is...

andypost’s picture

ah and probably we need a core's issue for hook_requirements() too

Wim Leers’s picture

Title: Document relation to Page Cache & Dynamic Page Cache modules » Initial README + document relation to Page Cache & Dynamic Page Cache modules
Status: Needs review » Fixed

I discussed this with @Fabianx. A hard dependency on dynamic_page_cache does not make sense, because it's totally possible to use BigPipe even without Dynamic Page Cache, and some sites will not want to use Dynamic Page Cache anyway.

Therefore hook_requirements() also seems excessive.

I wrote and committed an initial README, which also covers Page Cache & Dynamic Page Cache.

  • Wim Leers committed ec9a9e0 on 8.x-1.x
    Issue #2621394: Initial README + document relation to Page Cache...

  • Wim Leers committed 5d77cd6 on 8.x-1.x
    Follow-up for issue #2621394: improved documentation about how BigPipe...

  • Wim Leers committed 347dc94 on 8.x-1.x
    Follow-up for issue #2621394: whitespace fixes.
    

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.