- 🇺🇸United States smustgrave
This issue is being reviewed by the kind folks in Slack, #needs-review-queue-initiative. We are working to keep the size of Needs Review queue [2700+ issues] to around 400 (1 month or less), following Review a patch or merge request → as a guide.
Reroll looks good and issue is still addressed.
- Status changed to Needs work
about 2 years ago 1:51am 10 February 2023 - 🇳🇿New Zealand quietone
The issue summary is a statement of the problem. There is no indication of the proposed resolution.
I then went to apply the patch. I see that is against 9.4, which is in security support. The patch applied to 10.1.x so I tested it using the steps in the issue summary.
With the patch, it is now possible to save the basic sites settings page with an empty field for the default front page. I then looked at the system.site configuration.
$ ddev drush cget system.site _core: default_config_hash: VDJxTZtQR21qB4lvOq8zszJZLvLKrSPQpdn2E3T71Ww langcode: en uuid: 76eacedf-7cf4-4707-86c7-c478fe9ef9f6 name: dev2-web mail: v@example.com slogan: '' page: 403: '' 404: '' front: /user/login admin_compact_mode: false weight_select_max: 100 default_langcode: en
The front page path has not changed and we have the UI showing incorrect information. I then used the UI to change the the front page configuration to an empty string. I confirmed the change with drush
front: ''
. On navigating to the front page, I get a 404.I don't know if that is the intended behavior. But having the UI and the stored configuration out of sync is wrong.
I am adding a tag for an issue summary update and setting back to needs work.
- 🇮🇳India prem suthar Ahemdabad- Gujrat , Jodhpur - Rajsthan
Re-Roll the Patch For 10.1 by #41.
- 🇺🇸United States smustgrave
#46 was unnecessary as patch 41 still applied to D10 hiding patch.
Also please include an interdiff with all patches
- 🇬🇧United Kingdom longwave UK
I forgot that I commented on this issue before. I propose simplifying this setting and making the field required over in 📌 Make default front page setting required and remove special case of /user/login Fixed - this would mean this issue is no longer required.