Skip to content

Conversation

pallavigitwork
Copy link
Member

@pallavigitwork pallavigitwork commented Sep 5, 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.

Provides Fix for #2433

Description

The current IRC link of home page doesn't work. details in the Bug shared above.
This PR updates the link, which works, reference taken from the contributing document of main selenium github page.

Motivation and Context

home page links should be working.

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

  • Update IRC link to working web.libera.chat URL

Diagram Walkthrough

flowchart LR
  A["Old IRC Link"] -- "Replace" --> B["New Working IRC Link"]
  B --> C["Home Page Fixed"]
Loading

File Walkthrough

Relevant files
Bug fix
hugo.toml
Update IRC link URL                                                                           

website_and_docs/hugo.toml

  • Updated IRC URL from kiwiirc.com to web.libera.chat
  • Fixed broken link in developer links section
+1/-1     

Copy link

netlify bot commented Sep 5, 2025

👷 Deploy request for selenium-dev pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit d9cca94

Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes a broken IRC link on the home page by updating the URL from the non-functional kiwiirc.com client to the working web.libera.chat interface.

  • Updates the IRC link URL in the developer links section
  • Maintains the same IRC channel (#selenium) while switching to a functional web client

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Contributor

qodo-merge-pro bot commented Sep 5, 2025

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

🎫 Ticket compliance analysis 🔶

2433 - Partially compliant

Compliant requirements:

  • Fix the broken IRC link on the main Selenium website.
  • Change should be applied where the IRC link is defined in the site configuration/content.

Non-compliant requirements:

  • Ensure the updated link allows users to successfully access the IRC channel.

Requires further human verification:

  • Manually verify the updated IRC link works in browsers (e.g., Firefox on macOS and others) and properly loads the #selenium channel without errors.
⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ Recommended focus areas for review

Link Target

Confirm that https://web.libera.chat/#selenium is the correct and preferred URL (vs. alternative client URLs or direct irc:// scheme) and that it consistently joins the #selenium channel.

name = "IRC"
url = "https://web.libera.chat/#selenium"
icon = "fas fa-comments"

@pallavigitwork pallavigitwork requested a review from a team September 5, 2025 12:14
Copy link
Contributor

qodo-merge-pro bot commented Sep 5, 2025

PR Code Suggestions ✨

No code suggestions found for the PR.

@pallavigitwork pallavigitwork merged commit 6c26bdf into SeleniumHQ:trunk Sep 6, 2025
3 checks passed
@pallavigitwork pallavigitwork deleted the Fix2433 branch September 8, 2025 11:39
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