Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request adds OneTrust cookie consent scripts to the Docusaurus configuration. The review identifies a contradiction between the PR title and the implementation, and notes a potential domain mismatch for the OneTrust script ID which could lead to functional issues.
| // OneTrust Cookies Consent Notice for couchbase.com | ||
| headTags: [ | ||
| { | ||
| tagName: 'script', | ||
| attributes: { | ||
| src: 'https://cdn.cookielaw.org/scripttemplates/otSDKStub.js', | ||
| type: 'text/javascript', | ||
| charset: 'UTF-8', | ||
| 'data-domain-script': '748511ff-10bf-44bf-88b8-36382e5b5fd9', | ||
| }, | ||
| }, | ||
| { | ||
| tagName: 'script', | ||
| attributes: { | ||
| type: 'text/javascript', | ||
| }, | ||
| innerHTML: 'function OptanonWrapper() {}', | ||
| }, | ||
| ], |
There was a problem hiding this comment.
The pull request title "revert add banner" contradicts the code changes, which are adding the OneTrust cookie consent scripts. If the intention was to remove the banner, these lines should be deleted. If the intention was to add the banner, the PR title should be updated to reflect the correct action.
| locales: ['en'], | ||
| }, | ||
|
|
||
| // OneTrust Cookies Consent Notice for couchbase.com |
There was a problem hiding this comment.
The comment mentions "couchbase.com", which differs from the site's configured domain "ruby-cb-orm.com" (line 11). OneTrust configurations are typically domain-specific; using an ID intended for a different domain will prevent the consent banner from appearing or functioning correctly.
1 participant
No description provided.