Created new route for discord server and updated 2 discord links - #144
Conversation
There was a problem hiding this comment.
Pull request overview
This PR creates a new /discord/ redirect page and updates Discord invite links across configuration files from the old invite code to a new one (AR72D2nfpp).
- Created a new
/discord/route that redirects to the Discord server - Updated Discord invite links in
seo-config.jsonand_config.yml
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| discord.html | New redirect page created to handle /discord/ route with meta refresh and JavaScript redirect |
| seo-config.json | Updated Discord invite links in organization schema and contact point metadata |
| _config.yml | Updated Discord invite link in social media profiles configuration |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| --- | ||
| layout: default | ||
| permalink: /discord/ | ||
| redirect_to: https://discord.gg/AR72D2nfpp | ||
| --- | ||
| <!DOCTYPE html> | ||
| <html lang="es"> |
There was a problem hiding this comment.
The page uses both Jekyll front matter with layout: default and a full HTML document structure starting with <!DOCTYPE html>. This creates a duplicate layout. For a simple redirect, either remove the layout: default line from front matter (since Jekyll doesn't have a built-in redirect_to plugin) and keep the standalone HTML, or use a Jekyll redirect plugin. The current implementation will wrap the entire HTML document inside the default layout, creating invalid nested HTML.
| --- | ||
| layout: default | ||
| permalink: /discord/ | ||
| redirect_to: https://discord.gg/AR72D2nfpp |
There was a problem hiding this comment.
The redirect_to front matter parameter is not supported by Jekyll or GitHub Pages by default. This field will be ignored unless you install a plugin like jekyll-redirect-from. The actual redirect is handled by the meta refresh and JavaScript below, making this line non-functional. Either remove this line or install and configure jekyll-redirect-from plugin in _config.yml.
| redirect_to: https://discord.gg/AR72D2nfpp |
| <meta charset="UTF-8"> | ||
| <meta http-equiv="refresh" content="0; url=https://discord.gg/AR72D2nfpp"> | ||
| <meta name="robots" content="noindex, nofollow"> | ||
| <title>Redirigiendo a Discord...</title> |
There was a problem hiding this comment.
The title and fallback text are hardcoded in Spanish without i18n support. According to the project's Spanish-first approach with English translations, this page should use data-i18n attributes or translation modules to support both languages. Consider adding translation keys for 'Redirigiendo a Discord...' and the fallback link text.
| </script> | ||
| </head> | ||
| <body> | ||
| <p>Redirigiendo a Discord... <a href="https://discord.gg/AR72D2nfpp">Haz clic aquí si no eres redirigido automáticamente</a></p> |
There was a problem hiding this comment.
The fallback message is hardcoded in Spanish without i18n support. According to the project's i18n guidelines, this should use data-i18n attributes to support multiple languages. Consider adding translation modules to the front matter and using i18n keys for this text.
No description provided.