- Issue created by @sokru
- First commit to issue fork.
- Assigned to mparker17
- 🇨🇦Canada mparker17 UTC-4
I have been working on this, and have working code, so I'm going to assign this to myself.
- 🇨🇦Canada mparker17 UTC-4
Crediting @sukhi.chuhan for User Interface / User Experience help.
- Merge request !43Issue #3427092: Add an option to enable debug logging. → (Merged) created by mparker17
- Open on Drupal.org →Core: 10.2.1 + Environment: PHP 8.1 & MySQL 5.7last update
9 months ago Waiting for branch to pass - Open on Drupal.org →Core: 10.2.1 + Environment: PHP 8.1 & MySQL 5.7last update
9 months ago Waiting for branch to pass - Issue was unassigned.
- Status changed to Needs review
9 months ago 3:02pm 15 March 2024 - 🇨🇦Canada mparker17 UTC-4
This merge request is ready for review. Sorry for the delay, I was having some local issues with PHPUnit. Feedback is welcome.
Thanks again to @sukhi.chuhan!
- Status changed to RTBC
9 months ago 10:08pm 16 March 2024 - 🇨🇦Canada mparker17 UTC-4
Adding 🌱 Plan for 8.0.0-alpha release Active as a parent issue.
Also going to move this to RTBC for another maintainer to take a look at.
- Status changed to Needs work
9 months ago 8:16am 17 March 2024 - 🇫🇮Finland sokru
Thanks, looks good! I'd suggest we remove the "temp" and "temporary" terminology from MR. The word "debug" itself should indicate that its not meant to be used on production.
- 🇨🇦Canada mparker17 UTC-4
@sokru, thanks for the review!
@sukhi.chuhan, the UX design lead that I consulted with for the setting's text, advocated for using "temporary" to indicate that the setting should only be used for the short term, but she also okay with removing it if the word temporary feels duplicative for the audience (i.e.: admins changing ElasticSearch server settings).
I will push a commit shortly to remove it
- Open on Drupal.org →Core: 10.2.1 + Environment: PHP 8.1 & MySQL 5.7last update
9 months ago Waiting for branch to pass - Open on Drupal.org →Core: 10.2.1 + Environment: PHP 8.1 & MySQL 5.7last update
9 months ago Waiting for branch to pass - Open on Drupal.org →Core: 10.2.1 + Environment: PHP 8.1 & MySQL 5.7last update
9 months ago Waiting for branch to pass - Open on Drupal.org →Core: 10.2.1 + Environment: PHP 8.1 & MySQL 5.7last update
9 months ago Waiting for branch to pass - Status changed to RTBC
9 months ago 7:02pm 18 March 2024 - 🇨🇦Canada mparker17 UTC-4
Made all the requested changes, and fixed the PHPStan errors, so moving back to "RTBC"
- Open on Drupal.org →Core: 10.2.1 + Environment: PHP 8.1 & MySQL 5.7last update
9 months ago Waiting for branch to pass - Open on Drupal.org →Core: 10.2.1 + Environment: PHP 8.1 & MySQL 5.7last update
9 months ago Waiting for branch to pass -
sokru →
committed daa55ff9 on 8.0.x authored by
mparker17 →
Issue #3427092 by mparker17, sokru, sukhi.chuhan: Logging should be...
-
sokru →
committed daa55ff9 on 8.0.x authored by
mparker17 →
- Status changed to Fixed
9 months ago 7:59am 19 March 2024 Automatically closed - issue fixed for 2 weeks with no activity.