Skip to content

Conversation

@barbaraperic
Copy link
Contributor

@barbaraperic barbaraperic commented May 26, 2025

Changes

  1. Added essential SEO elements:
    • Meta description
    • robots.txt file
    • sitemap.xml file
  2. Improved accessibility by enhancing image alt attributes in Footer and Header components

Purpose

These changes aim to improve the website's search engine visibility and make it more accessible to users relying on screen readers or other assistive technologies.

Testing

Verify:
• The sitemap.xml is properly generated and accessible
• robots.txt is correctly configured
• Image alt texts are meaningful and descriptive in Footer and Header components
• Meta descriptions appear correctly in page source

Note

I added a slight refactor in the Header component so that the elements are grouped together and aligned with the main content.

Screenshot 2025-06-26 at 13 55 53

… and sitemap.xml; improve image alt attributes in Footer and Header components
… utility function for conditional class names
@barbaraperic barbaraperic force-pushed the bp/fix-lighthouse-issues branch from 5c6a892 to 3fd14d1 Compare June 26, 2025 12:51
@barbaraperic
Copy link
Contributor Author

@ianconsolata tagging for visibility :)

@smagdali
Copy link

smagdali commented Jul 2, 2025

shouldn't this be "Disallow: /" for vote.storswift.io (as the dev site) and only "Allow: /" on vote.fil.org ?

Also shouldn't the sitemap change similarly?

@barbaraperic
Copy link
Contributor Author

shouldn't this be "Disallow: /" for vote.storswift.io (as the dev site) and only "Allow: /" on vote.fil.org ?

Also shouldn't the sitemap change similarly?

Ah thank you, didn't realised there were 2 different deployments. Looking at the code, I believe that vote.fil.org is a test environment and vote.storeswift is the production? href={network === 'testnet' ? "https://vote.fil.org/" : "https://vote.storswift.io/"}

So we should have something like this?

User-agent: *
Disallow: /api/
Disallow: /power_voting/api/
Allow: /

Disallow: https://vote.fil.org/
Sitemap: https://vote.storswift.io/sitemap.xml

@smagdali
Copy link

smagdali commented Jul 2, 2025

shouldn't this be "Disallow: /" for vote.storswift.io (as the dev site) and only "Allow: /" on vote.fil.org ?
Also shouldn't the sitemap change similarly?

Ah thank you, didn't realised there were 2 different deployments. Looking at the code, I believe that vote.fil.org is a test environment and vote.storeswift is the production? href={network === 'testnet' ? "https://vote.fil.org/" : "https://vote.storswift.io/"}

vote.fil.org should be production. @ianconsolata, unless I'm insane?

So we should have something like this?

User-agent: *
Disallow: /api/
Disallow: /power_voting/api/
Allow: /

Disallow: https://vote.fil.org/
Sitemap: https://vote.storswift.io/sitemap.xml

@barbaraperic
Copy link
Contributor Author

@ianconsolata following up on this one :)

@ianconsolata
Copy link
Collaborator

Yes, vote.fil.org is production.

Allow: /

# Sitemap location
Sitemap: https://vote.storswift.io/sitemap.xml
Copy link
Collaborator

Choose a reason for hiding this comment

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

vote.fil.org

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