Skip to content

Offline search: fingerprint data in production only, take 2#714

Merged
LisaFC merged 1 commit intogoogle:masterfrom
chalin:chalin-offline-srch-fingerprint-in-prod-only-2021-10-06
Oct 8, 2021
Merged

Offline search: fingerprint data in production only, take 2#714
LisaFC merged 1 commit intogoogle:masterfrom
chalin:chalin-offline-srch-fingerprint-in-prod-only-2021-10-06

Conversation

@chalin
Copy link
Copy Markdown
Collaborator

@chalin chalin commented Oct 6, 2021

I tested this with and without offline search enabled, in development and production modes and it compiles fine; fingerprints in production mode, and doesn't otherwise.

/cc @celestehorgan @nate-double-u

@google-cla google-cla Bot added the cla: yes Indicates the PR's author has signed the CLA. label Oct 6, 2021
@chalin
Copy link
Copy Markdown
Collaborator Author

chalin commented Oct 6, 2021

I tested with both Hugo 0.82.0 and 0.88.1.

@tekezo
Copy link
Copy Markdown
Contributor

tekezo commented Oct 6, 2021

LGTM, thanks!

@LisaFC
Copy link
Copy Markdown
Collaborator

LisaFC commented Oct 8, 2021

Excellent, thanks!

@LisaFC LisaFC merged commit c93142e into google:master Oct 8, 2021
@chalin chalin deleted the chalin-offline-srch-fingerprint-in-prod-only-2021-10-06 branch October 15, 2021 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes Indicates the PR's author has signed the CLA.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PR #710 reverted #691, so the latter needs to be reapplied

3 participants