Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add SEO tags (#745) #759

Merged
merged 1 commit into from
May 22, 2024
Merged

feat: add SEO tags (#745) #759

merged 1 commit into from
May 22, 2024

Conversation

EtayZaslavsky
Copy link
Collaborator

Description

Added SEO tags to index.html. added an image version of the svg logo for social networks' SEO.
Please review @NoamGaash :)

@R894 R894 linked an issue May 21, 2024 that may be closed by this pull request
6 tasks
Copy link
Collaborator

@R894 R894 left a comment

Choose a reason for hiding this comment

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

Looks good to me!

Below are some suggestions I think might help us

If we want to take it a step further its also possible to add schema.org tags

  <script type="application/ld+json">
  {
    "@context": "http://schema.org",
    "@type": "Website",
    "name": "דאטאבוס",
    "url": "https://open-bus-map-search.hasadna.org.il/",
    "description": "אתר דאטאבוס מציג נתונים על איכות קווי התחבורה הציבורית בארץ, מתוך מקורות מידע ממשלתיים.",
    "sameAs": [
      "https://www.facebook.com/hasadna.org.il/",
      "https://x.com/hasadna"
    ]
  }
  </script>

After running the site through an SEO checker it also suggested using header tags ( <h1> - <h6> ), maybe we an add that somewhere also

Copy link
Member

@NoamGaash NoamGaash left a comment

Choose a reason for hiding this comment

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

Thank you! It looks great, and solve 99% of the problem. Looking forward to see it merged 🥳
Let's leave the issue open, as we can still use some more contributions regarding canonical links and a sitemap

<title>דאטאבוס</title>
<meta name="title" content='דאטאבוס - נתוני ביצוע לתחב"צ הישראלי' />
<meta name="description"
content="אתר דאטאבוס מציג נתונים על איכות קווי התחבורה הציבורית בארץ, מתוך מקורות מידע ממשלתיים, במטרה להביא תמונת מצב על תפקוד התחבורה הציבורית בארץ." />
Copy link
Member

Choose a reason for hiding this comment

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

🚀

Copy link
Member

Choose a reason for hiding this comment

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

That's a weird version of the logo, but why not? I like it.

@NoamGaash NoamGaash removed a link to an issue May 21, 2024
6 tasks
@NoamGaash NoamGaash mentioned this pull request May 21, 2024
6 tasks
@EtayZaslavsky
Copy link
Collaborator Author

Great!
@NoamGaash - Will be onto the other tasks in few days or so.
@R894 - I will definitely add schema.org tags.

@EtayZaslavsky EtayZaslavsky merged commit 9facbce into main May 22, 2024
17 checks passed
@EtayZaslavsky EtayZaslavsky deleted the feat/add-seo-tags branch May 22, 2024 21:42
@NoamGaash
Copy link
Member

Thank you! 🥳

@all-contributors please add @EtayZaslavsky for code

Copy link
Contributor

@NoamGaash

I've put up a pull request to add @EtayZaslavsky! 🎉

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