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

3.0.0 #59

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

3.0.0 #59

wants to merge 7 commits into from

Conversation

samdark
Copy link
Owner

@samdark samdark commented May 14, 2018

Refactored library to be more flexible.

TODO

  • Implement video extension
  • Implement image extension

@b1rdex
Copy link

b1rdex commented May 14, 2018

About #14: you implemented just alternate language pages. There is also alternate mobile pages I mentioned in original issue — https://developers.google.com/search/mobile-sites/mobile-seo/separate-urls#annotation-in-sitemaps

@evannoronha
Copy link

@samdark is there a reason this pull wasn't merged? We want to use this feature for our sitemap generation.

@samdark
Copy link
Owner Author

samdark commented Jun 26, 2019

It isn't finished.

@evannoronha
Copy link

Do you plan on finishing it?

@samdark
Copy link
Owner Author

samdark commented Jun 27, 2019

Yes but it's frozen now due to lack of time.

Google can't properly read sitemap files with `_` in their name
php7.4 - mime type seems to change to application/gzip rather than application/x-grip
file names not correct.
@wolfgangzwiauer
Copy link

Hey - what is missing for this to be merged/released? How can we help?

@samdark
Copy link
Owner Author

samdark commented Aug 11, 2023

  1. Mege conflicts.
  2. Video extensions.
  3. Image extensions.

You can fork this branch and do your own pull request.

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.

7 participants