- Issue created by @gbyte
- 🇮🇳India kkalashnikov Ghaziabad, India
kunalgautam → made their first commit to this issue’s fork.
- 🇺🇸United States thejimbirch Cape Cod, Massachusetts
@kunalgautam
simple_sitemap_engines is a submodule of drupal/simple_sitemap.
This line in the composer.json file should be removed as it is not needed. The submodule gets required when drupal/simple_sitemap is required.
"drupal/simple_sitemap_engines": "^4.2.2",@gbyte How is the key generated, and how is it saved? I assume we can use config actions in a recipe to save it, but not sure how a recipe could generate the key.
- 🇩🇪Germany gbyte Berlin
@gbyte How is the key generated, and how is it saved? I assume we can use config actions in a recipe to save it, but not sure how a recipe could generate the key.
After module installation
The IndexNow functionality is not available as long as the below requirement is not met.
Requirement warning
Another warning on the module's settings page
After key generation
Klicking 'Generate verification key' saves a new key to Drupal state. We generate the key through
\Drupal::service('uuid')->generate().A nudge to move the key to settings.php
After moving the key to settings.php
A nudge and facility to delete the key from state
I think this explains the process. I think the easiest and least invasive would be to generate the key and put it into state with
SimpleSitemapEnginesForm::generateKey()- a user's key in settings.php would override that automatically generated key anyway.The upside of an automatically generated key would be the user not needing to check requirements and pressing a button.
The downside being dev environments would start sending index requests to IndexNow.If we go ahead with automatic generation, we have to decide if the upstream module should generate that key on module install or if Drupal CMS should do it.
What do you think?