Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adds phpMyAdmin to MySql component #2055

Merged
merged 3 commits into from
Feb 3, 2024
Merged

Adds phpMyAdmin to MySql component #2055

merged 3 commits into from
Feb 3, 2024

Conversation

timheuer
Copy link
Member

@timheuer timheuer commented Feb 2, 2024

This adds the phpMyAdmin container image when WithPhpMyAdmin is used on a MySql container.

Fixes #1936

Why not adminer?
Adminer doesn't support very well multiple host configurations nor the docker internal host names very well in my testing.

Single MySqlInstance (automatic login):

singleinstance

Multiple instances (would show a drop-down in phpMyAdmin and then would have to login)

multipleinstances

Microsoft Reviewers: Open in CodeFlow

This adds the phpMyAdmin container image when `WithPhpMyAdmin` is used on a MySql container.
@dotnet-issue-labeler dotnet-issue-labeler bot added the area-codeflow for labeling automated codeflow. intentionally a different color! label Feb 2, 2024
@timheuer timheuer requested a review from mitchdenny February 2, 2024 23:17
@mitchdenny
Copy link
Member

This is looking pretty good. One of the things that I wanted to do for Postgres but didn't get to in P3 was make it so that if you do WithPgAdmin(...) on one resource, but not the other, you don't get the server registered in the PG admin container. This code has the same issue but I wouldn't block it for that.

@timheuer timheuer marked this pull request as ready for review February 3, 2024 04:04
@timheuer timheuer requested a review from davidfowl February 3, 2024 04:04
Copy link
Member

@davidfowl davidfowl left a comment

Choose a reason for hiding this comment

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

I have small nit comments but great work @timheuer !

@davidfowl davidfowl merged commit 73b9e0b into main Feb 3, 2024
8 checks passed
@davidfowl davidfowl deleted the phpymyadmin branch February 3, 2024 05:36
radical pushed a commit to radical/aspire that referenced this pull request Feb 6, 2024
* Adds phpMyAdmin to MySql component

This adds the phpMyAdmin container image when `WithPhpMyAdmin` is used on a MySql container.

* Added tests

* Added mysql playground
@github-actions github-actions bot locked and limited conversation to collaborators Apr 22, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-codeflow for labeling automated codeflow. intentionally a different color!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for adminer for mysql
3 participants