Skip to content

Test: instant cookieyes#2705

Closed
claudiacodacy wants to merge 3 commits into
masterfrom
test-instant-cookieyes
Closed

Test: instant cookieyes#2705
claudiacodacy wants to merge 3 commits into
masterfrom
test-instant-cookieyes

Conversation

@claudiacodacy

@claudiacodacy claudiacodacy commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

👀 Live preview

🚧 To do

  • If relevant, include the Jira issue key at the end of the pull request title
  • Perform a self-review of the changes
  • Fix any issues reported by the CI/CD

claudiacodacy and others added 2 commits July 16, 2026 12:10
@claudiacodacy @claude
test: reproduce navigation.instant + CookieYes on preview (do not merge)
e477b2a 
Re-enables navigation.instant and injects the production CookieYes script
into <head> so the real banner + auto-blocking can be exercised against
instant navigation on a Netlify preview (CookieYes is normally GTM-injected
and scoped to docs.codacy.com, so it never fires on preview hosts).

Throwaway branch for diagnosing the unstyled-on-navigation issue. Not for master.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@claudiacodacy
replace cookie tag to test staging
ad947d0
@codacy-production

codacy-production Bot commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

AI Reviewer: first review requested successfully. AI can make mistakes. Always validate suggestions.

Run reviewer

TIP This summary will be updated as you push new changes.

@github-actions
github-actions Bot temporarily deployed to Netlify July 16, 2026 11:19 Inactive
gemini-code-assist[bot]
gemini-code-assist Bot reviewed Jul 16, 2026
View reviewed changes

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request enables the navigation.instant feature in the MkDocs configuration and introduces a test-only CookieYes integration script to verify its behavior on preview environments. Feedback points out a typo in the CookieYes script source URL, which is missing the leading 'h' in https://.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread theme/partials/integrations/cookieyes.html Outdated
host (banner enabled + auto-blocking), NOT the production key that GTM
injects on docs.codacy.com. Do NOT merge this to master. #}
<!-- Start cookieyes banner -->
<script id="cookieyes" type="text/javascript" src="ttps://cdn-cookieyes.com/client_data/c5874f8d39a04e1ffb9c4b1c8b3074e9/script.js"></script>

@gemini-code-assist gemini-code-assist Bot Jul 16, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

high

The script URL protocol is missing the leading 'h' (ttps:// instead of https://). This will prevent the CookieYes banner script from loading. Please correct it to https://.

Suggested change
<script id="cookieyes" type="text/javascript" src="ttps://cdn-cookieyes.com/client_data/c5874f8d39a04e1ffb9c4b1c8b3074e9/script.js"></script>
<script id="cookieyes" type="text/javascript" src="https://cdn-cookieyes.com/client_data/c5874f8d39a04e1ffb9c4b1c8b3074e9/script.js"></script>

@github-actions
github-actions Bot temporarily deployed to Netlify July 16, 2026 11:26 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
@gemini-code-assist gemini-code-assist[bot] gemini-code-assist[bot] left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@claudiacodacy