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

Add mention of phpMyAdmin and the Sequel Pro command #793

Merged
merged 2 commits into from
Apr 19, 2018

Conversation

nearlythere
Copy link
Contributor

The Problem/Issue/Bug:

Tell users about database management options with ddev

See #769

How this PR Solves The Problem:

Add information about the phpMyAdmin service included in ddev.
Add explanation of sequelpro command

Manual Testing Instructions:

Visit
https://ddev.readthedocs.io/en/latest/users/developer-tools/.

Automated Testing Overview:

Related Issue Link(s):

#769

Release/Deployment notes:

Copy link
Member

@rfay rfay left a comment

Choose a reason for hiding this comment

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

Thanks for this, it will be a good improvement, minor changes requested.

http://mysite.ddev.local:8036
```

If you use [Sequel Pro](https://www.sequelpro.com/) for macOS, run `ddev sequelpro` within a project folder, and Sequel Pro will launch and load the database for that project. Sequel Pro is a free open source database browser for macOS.
Copy link
Member

Choose a reason for hiding this comment

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

Maybe "access the database" would be better than "load the database"? Certainly it's not loading a db.

Probably better to just say "If you use the free Sequel Pro..." and drop the last sentence.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

k! sorted.

@@ -23,6 +23,18 @@ http://mysite.ddev.local:8025

Please note this will not intercept emails if your application is configured to use SMTP or a 3rd-party ESP integration. If you are using SMTP for outgoing mail handling ([Swiftmailer](https://www.drupal.org/project/swiftmailer) or [SMTP](https://www.drupal.org/project/smtp) modules for example), update your application configuration to use `localhost` on port `1025` as the SMTP server locally in order to use MailHog.

### Database Management

[phpMyAdmin](https://www.phpmyadmin.net/) is a free software tool to manage MySQL and MariaDB databases from a browser. phpMyAdmin comes installed with ddev.
Copy link
Member

Choose a reason for hiding this comment

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

Maybe mention that ddev describe gets all this info? It's not directly related, but it's a much faster way to find things out.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added a mention to the top of this section on tools.

Copy link
Contributor

@rickmanelius rickmanelius left a comment

Choose a reason for hiding this comment

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

Confirmed the accuracy of the information re:ports and I think this looks good to go. Good catch by @rfay and we need to either keep pushing people to use ddev describe or just output it by default after spin up :)

Copy link
Member

@rfay rfay left a comment

Choose a reason for hiding this comment

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

Looks good to me, thanks!

@rfay rfay merged commit 98bcc81 into ddev:master Apr 19, 2018
@rickmanelius rickmanelius modified the milestone: v0.18.0 May 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants