Skip to content

Conversation

@rpallavisharma
Copy link
Member

@rpallavisharma rpallavisharma commented Aug 16, 2025

User description

Thanks for contributing to the Selenium site and documentation!
A PR well described will help maintainers to review and merge it quickly

Before submitting your PR, please check our contributing guidelines.
Avoid large PRs, and help reviewers by making them as simple and short as possible.

fixed extra documentation link bug in left menu

Description

fixed extra documentation link bug in left menu

Motivation and Context

fixed extra documentation link bug in left menu

Types of changes

  • Change to the site (I have double-checked the Netlify deployment, and my changes look good)
  • Code example added (and I also added the example to all translated languages)
  • Improved translation
  • Added new translation (and I also added a notice to each document missing translation)

Checklist

  • I have read the contributing document.
  • I have used hugo to render the site/docs locally and I am sure it works.

PR Type

Bug fix


Description

  • Remove unused 'other' language configuration from Hugo

  • Delete orphaned documentation index file for 'other' language


Diagram Walkthrough

flowchart LR
  A["Hugo Config"] -- "remove commented" --> B["Clean Language Settings"]
  C["Other Language Index"] -- "delete file" --> D["Fixed Menu Structure"]
Loading

File Walkthrough

Relevant files
Bug fix
_index.other.md
Delete unused other language documentation index                 

website_and_docs/content/documentation/_index.other.md

  • Delete entire file containing documentation index for 'other' language
  • Remove title, linkTitle, and translation notice content
+0/-15   
Configuration changes
hugo.toml
Remove commented other language configuration                       

website_and_docs/hugo.toml

  • Remove commented out 'other' language configuration block
  • Clean up unused language settings and parameters
+0/-6     

@rpallavisharma rpallavisharma self-assigned this Aug 16, 2025
@netlify
Copy link

netlify bot commented Aug 16, 2025

👷 Deploy request for selenium-dev pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 74ae226

@qodo-merge-pro
Copy link
Contributor

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ Recommended focus areas for review

Context Check

Ensure removing the commented 'other' language block and deleting the 'other' index file does not leave any remaining references (menus, params, i18n, or links) that expect the 'other' language, to avoid broken menu entries.

# Import docsy theme as module
[module]
  proxy = "direct"

@qodo-merge-pro
Copy link
Contributor

PR Code Suggestions ✨

No code suggestions found for the PR.

@rpallavisharma rpallavisharma requested a review from diemol August 16, 2025 13:27
@rpallavisharma
Copy link
Member Author

@diemol , requesting your review for this. thanks.

Copy link
Member

@diemol diemol left a comment

Choose a reason for hiding this comment

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

Thank you.

@diemol diemol merged commit 900ea71 into SeleniumHQ:trunk Aug 16, 2025
3 checks passed
@rpallavisharma
Copy link
Member Author

Thanks Diego

@rpallavisharma rpallavisharma deleted the fixBug-pal branch August 18, 2025 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants