Skip to content

Conversation

@ArmanTaheriGhaleTaki
Copy link

Summary

This PR fixes a malformed GitHub URL that contained an erroneous double slash (//) in the path. The incorrect URL structure was causing potential resolution issues and follows best practices for clean, canonical URLs.

Changes Made

Fixed URL path: Changed https://github.com/apache/kafka-site//tree/markdown/
image

to https://github.com/apache/kafka-site/tree/markdown/...

image

Removed duplicate slash: Eliminated the redundant // that appeared before tree in the URL path

and there is another change

The kafka-site repository has disabled its GitHub Issues section, making the original link non-functional for users trying to submit feedback. All Apache Kafka project issues are now centrally managed through the official Apache JIRA instance, which is the standard issue tracking system for Apache Software Foundation projects.

Copy link
Member

@mimaison mimaison left a comment

Choose a reason for hiding this comment

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

Thanks for the PR. I'm not sure it's addressing the issue properly.

# The responses that the user sees after clicking "yes" (the page was helpful) or "no" (the page was not helpful).
'yes': >-
Glad to hear it! Please <a href="https://github.com/apache/kafka-site/issues/new">tell us how we can improve</a>.
Glad to hear it! Please <a href="https://issues.apache.org/jira/browse/KAFKA">tell us how we can improve</a>.
Copy link
Member

Choose a reason for hiding this comment

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

I think this feature is disabled as the website does not have Google Analytics. So we may just be able to remove this configuration.

Copy link
Member

Choose a reason for hiding this comment

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

We have a Create documentation issue link on all pages that has links to Github issues. This would need updating but I'm not sure where it's configured though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants