Skip to content

fix SEO info in HTML#10

Merged
NoEgAm merged 5 commits into
mainfrom
seo
May 28, 2025
Merged

fix SEO info in HTML#10
NoEgAm merged 5 commits into
mainfrom
seo

Conversation

@tiye
Copy link
Copy Markdown
Member

@tiye tiye commented May 26, 2025

No description provided.

@tiye tiye requested review from a team and Copilot May 26, 2025 17:34
Copy link
Copy Markdown
Contributor

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 adds SEO and social media meta tags to the main HTML page and updates the Node.js version in the CI workflow.

  • Extended <head> in index.html with <title>, description, keywords, author, Open Graph, Twitter cards, and favicon.
  • Bumped Node.js version from 22 to 24.x in the GitHub Actions workflow.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.

File Description
index.html Added SEO meta tags, social previews (OG & Twitter), and favicon link.
.github/workflows/upload.yaml Updated node-version from 22 to 24.x for CI environments.
Comments suppressed due to low confidence (1)

index.html:25

  • [nitpick] Consider moving the viewport meta tag directly after the charset declaration for improved mobile compatibility.
<meta name="viewport" content="width=device-width, initial-scale=1, maximum-scale=1, minimum-scale=1, user-scalable=no"/>

Comment thread index.html Outdated
Comment thread index.html Outdated
Comment thread index.html Outdated
Comment thread index.html Outdated
Comment thread index.html Outdated
tiye and others added 4 commits May 28, 2025 01:50
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
@NoEgAm NoEgAm merged commit fc0581e into main May 28, 2025
1 check passed
@NoEgAm NoEgAm deleted the seo branch May 28, 2025 07:21
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.

3 participants