-
Notifications
You must be signed in to change notification settings - Fork 370
[Website] Database panel with Adminer & phpMyAdmin #2926
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
Conversation
ad80c72 to
be73ee4
Compare
...ges/playground/website/src/components/site-manager/site-database-panel/phpmyadmin-button.tsx
Show resolved
Hide resolved
@adamziel Strange. Looks like the Adminer installation step worked, but somehow the extensions were not installed or applied. Can you replicate it reliably and also in an incognito window? I wonder why for me and in the tests it works. |
|
@adamziel Oh, I can actually replicate it. It happens when I open phpMyAdmin first and then Adminer. Checking 👀 |
0fecef8 to
2f258c0
Compare
|
@adamziel The Adminer problem should be fixed now. Also rebased on trunk. |
|
@JanJakes In PhpMyAdmin, the edit button is there, but leads to the following error on my end
|
|
Executing |
|
I think we're good to merge this if the UI says it's an "early access" experimental feature. I'll see if I can make the change. |
@adamziel I just noticed that myself today, checking what the problem is.
Maybe I'm ignoring errors "too much". Both Adminer and phpMyAdmin run some statements that we don't support, and they work fine if we return an empty resultset in those cases. Maybe it can silence too many errors, and I should probably explicitly list only the failures we want to ignore 🤔
Oh, this looks like a regression I introduced with the last refactor. I can fix that quickly, I think.
Are you looking at this one, or should I add it as well? |
|
@JanJakes I've added a notice to the UI, I'm poking around a few visual elements and after that I think we can merge. |
|
@adamziel Thanks! I also have a fix for the |
…ge outside of the flex container
|
Docs failure seems like a flaky test unrelated to this PR, let's address it separately. |



Motivation for the change, related issues
Ships a Database panel with Adminer & phpMyAdmin:
Screen.Recording.2025-11-26.at.11.34.00.mov
Implementation details
SQLITE_ENABLE_COLUMN_METADATAflag, and phpMyAdmin cannot determine which columns are unique. To be fixed in a separate PR.Testing Instructions (or ideally a Blueprint)
npx nx dev playground-website.