Skip to content
This repository has been archived by the owner on Jun 8, 2021. It is now read-only.

Feature/sitemapindex #4

Open
wants to merge 2 commits into
base: release/4.x.x
Choose a base branch
from

Conversation

encreinformatique
Copy link

This Pull Request add the feature for sitemapindex.

The ReadMe has been modified to explain the new feature.

The start option has been added. It starts automatically the element
urlset. When set to false, it disable the startElement and needs a
manual declaration with the method startElement.
This method has been added to the SitemapGenerator.

The methods addEntry and addSitemap now returns the Class to have the
posibility to chain the entries together. (See the example into the
readme.)

A test has been added to the feature. I am not really familiar yet with Behat.
As PHPunit and PHPspecs are not implemented, I did not add those tests.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
The ReadMe has been modified to explain the new feature.

The start option has been added. It starts automatically the element
urlset. When set to false, it disable the startElement and needs a
manual declaration with the method startElement.
This method has been added to the SitemapGenerator.

The methods addEntry and addSitemap now returns the Class to have the
posibility to chain the entries together. (See the example into the
readme.)

A test has been added to the feature. I am not familiar yet with the
library used.
As PHPunit and PHPspecs are not implemented, I did not add those tests.
The tests have been adjusted for Behat after testing the project.

The branch passed the tests.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant