Skip to content

Conversation

@TC-MO
Copy link
Contributor

@TC-MO TC-MO commented Dec 8, 2025

Note

Removes Giscus comments from doc pages and drops related config, styles, and dependency.

  • Theme (doc item layout):
    • Remove Giscus component, wrapper, and display logic from src/theme/DocItem/Layout/index.js.
    • Simplify imports accordingly.
  • Config:
    • Delete customFields.forbiddenGiscusDocRegExpStrings from docusaurus.config.js.
  • Styles:
    • Remove .giscus CSS from src/theme/DocItem/Layout/styles.module.css.
  • Dependencies:
    • Remove @giscus/react from package.json and related entries from package-lock.json.

Written by Cursor Bugbot for commit f209f80. Configure here.

@TC-MO TC-MO requested a review from B4nan December 8, 2025 11:54
@TC-MO TC-MO self-assigned this Dec 8, 2025
@TC-MO TC-MO added documentation Improvements or additions to documentation. t-docs Issues owned by technical writing team. labels Dec 8, 2025
@apify-service-account
Copy link

Preview for this PR was built for commit 077a291 and is ready at https://pr-2133.preview.docs.apify.com!

@TC-MO
Copy link
Contributor Author

TC-MO commented Dec 8, 2025

@B4nan Is there anything else to be done to remove giscus (apart from removing unused imports and make lint check happy?)

Copy link
Member

@B4nan B4nan left a comment

Choose a reason for hiding this comment

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

Should be just about that. You just need to drop the unused imports to make the linter happy.

@apify-service-account
Copy link

Preview for this PR was built for commit f209f80 and is ready at https://pr-2133.preview.docs.apify.com!

@TC-MO TC-MO marked this pull request as ready for review December 8, 2025 13:04
@TC-MO TC-MO merged commit ee975fb into master Dec 10, 2025
13 checks passed
@TC-MO TC-MO deleted the remove-giscus branch December 10, 2025 09:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation. t-docs Issues owned by technical writing team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants