Skip to content

Conversation

@rfay
Copy link
Member

@rfay rfay commented Nov 7, 2025

The Issue

Show description/user/pass/base image in ddev describe

image

Manual Testing Instructions

ddev add-on get https://github.com/ddev/ddev-solr/tarball/refs/pull/73/head
ddev restart

Automated Testing Overview

Release/Deployment Notes

  • This was done as a demonstration of this new feature at TYPO3Camp RheinRuhr 2025 live in class :)
  • Changed ddev_version_constraint to v1.24.10

@rfay rfay requested a review from stasadev November 7, 2025 12:51
@rfay
Copy link
Member Author

rfay commented Nov 7, 2025

@davereid @gitressa take a look

Copy link
Member

@stasadev stasadev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

And we need to include x-ddev change here https://github.com/ddev/ddev-solr#changing-solr-password to show new password

-        Pass: SolrRocks
+        Pass: SolrRocksSolrRocks

Co-authored-by: Stanislav Zhuk <stasadev@gmail.com>
@rfay rfay marked this pull request as draft November 7, 2025 14:08
@gitressa
Copy link
Contributor

gitressa commented Nov 7, 2025

Thanks, this will be a great feature! And I agree with slimming the width, it's a good call

@rfay
Copy link
Member Author

rfay commented Nov 8, 2025

Moved to draft; I think it can use a tiny bit of test coverage.

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.

4 participants