Add shortcut assistant privacy policy - #39
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedAuto incremental reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
WalkthroughThis PR adds a new privacy policy page for the Shortcut Assistant Chrome extension. It introduces two React components: a ChangesShortcut Assistant Privacy Policy Feature
🎯 1 (Trivial) | ⏱️ ~3 minutes 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@src/app/shortcut-assistant/privacy-policy/page.tsx`:
- Around line 6-8: The wrapper div in page.tsx uses "container mx-auto px-6
sm:py-16" while the PrivacyPolicy component's root <section> in
privacy-policy.tsx also uses "container", causing nested Tailwind container
conflicts; remove the duplicate by deleting the "container" class from the
wrapper div in src/app/shortcut-assistant/privacy-policy/page.tsx (keep spacing
utilities like mx-auto px-6 sm:py-16 if needed) so only the PrivacyPolicy
component's <section> retains the container behavior.
In `@src/app/shortcut-assistant/privacy-policy/privacy-policy.tsx`:
- Around line 18-19: Remove the redundant <strong> tags nested inside <h3
className="font-bold ..."> headings in the privacy-policy component: for example
the <h3> that wraps "Introduction" and the other <h3> headings that similarly
wrap their text (the ones around the "Introduction" heading and the other
section titles). Edit the JSX in privacy-policy.tsx to delete the inner <strong>
elements and leave the text directly inside the <h3 className="font-bold..."> so
the heading styling is provided by the class only; ensure no other markup
changes and run a quick UI check to confirm layout/spacing remains correct.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro
Run ID: 2ad7a8b3-210c-4cb0-ae24-b3120a2e3efa
📒 Files selected for processing (2)
src/app/shortcut-assistant/privacy-policy/page.tsxsrc/app/shortcut-assistant/privacy-policy/privacy-policy.tsx
* Add shortcut assistant privacy policy (#39) * Add shortcut assistant privacy policy * Refactor Privacy Policy formatting and remove unnecessary strong tags * Increment version (#40) * Add dependabot config (#42) * Bump next from 16.1.2 to 16.2.9 (#51) Bumps [next](https://github.com/vercel/next.js) from 16.1.2 to 16.2.9. - [Release notes](https://github.com/vercel/next.js/releases) - [Changelog](https://github.com/vercel/next.js/blob/canary/release.js) - [Commits](vercel/next.js@v16.1.2...v16.2.9) --- updated-dependencies: - dependency-name: next dependency-version: 16.2.9 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump @eslint/eslintrc from 3.3.1 to 3.3.5 (#50) Bumps [@eslint/eslintrc](https://github.com/eslint/eslintrc) from 3.3.1 to 3.3.5. - [Release notes](https://github.com/eslint/eslintrc/releases) - [Changelog](https://github.com/eslint/eslintrc/blob/main/CHANGELOG.md) - [Commits](eslint/eslintrc@v3.3.1...eslintrc-v3.3.5) --- updated-dependencies: - dependency-name: "@eslint/eslintrc" dependency-version: 3.3.5 dependency-type: direct:development update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump @tailwindcss/postcss from 4.1.15 to 4.3.1 (#48) Bumps [@tailwindcss/postcss](https://github.com/tailwindlabs/tailwindcss/tree/HEAD/packages/@tailwindcss-postcss) from 4.1.15 to 4.3.1. - [Release notes](https://github.com/tailwindlabs/tailwindcss/releases) - [Changelog](https://github.com/tailwindlabs/tailwindcss/blob/main/CHANGELOG.md) - [Commits](https://github.com/tailwindlabs/tailwindcss/commits/v4.3.1/packages/@tailwindcss-postcss) --- updated-dependencies: - dependency-name: "@tailwindcss/postcss" dependency-version: 4.3.1 dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Add checkmark (#56) * Bump eslint-plugin-jest from 28.12.0 to 29.15.2 (#55) Bumps [eslint-plugin-jest](https://github.com/jest-community/eslint-plugin-jest) from 28.12.0 to 29.15.2. - [Release notes](https://github.com/jest-community/eslint-plugin-jest/releases) - [Changelog](https://github.com/jest-community/eslint-plugin-jest/blob/main/CHANGELOG.md) - [Commits](jest-community/eslint-plugin-jest@v28.12.0...v29.15.2) --- updated-dependencies: - dependency-name: eslint-plugin-jest dependency-version: 29.15.2 dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump @eslint/js from 9.27.0 to 10.0.1 (#54) Bumps [@eslint/js](https://github.com/eslint/eslint/tree/HEAD/packages/js) from 9.27.0 to 10.0.1. - [Release notes](https://github.com/eslint/eslint/releases) - [Commits](https://github.com/eslint/eslint/commits/v10.0.1/packages/js) --- updated-dependencies: - dependency-name: "@eslint/js" dependency-version: 10.0.1 dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump tailwind-merge from 3.3.1 to 3.6.0 (#53) Bumps [tailwind-merge](https://github.com/dcastil/tailwind-merge) from 3.3.1 to 3.6.0. - [Release notes](https://github.com/dcastil/tailwind-merge/releases) - [Commits](dcastil/tailwind-merge@v3.3.1...v3.6.0) --- updated-dependencies: - dependency-name: tailwind-merge dependency-version: 3.6.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Summary by CodeRabbit