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

Updating Vagrant section to MAMP/WAMP/VVV #148

Merged
merged 19 commits into from
Oct 4, 2016
Merged

Conversation

rclations
Copy link
Contributor

Changes

  • Updated the vagrant setup documentation to recommend MAMP, WAMP, and VVV, for simpler on boarding for users of all types.
  • Updated vagrant setup documentation to direct users to the Largo documentation (instead of deploy-tools), centralizing the documentation on Vagrant setup.

Why?

We needed a simpler option for new users to get started with development, without having to setup a custom Vagrant build. MAMP & WAMP offer a simple alternative, while VVV streamlines our existing Vagrant process for advanced users and takes care of a lot of the inherent bugs.

Benefits of VVV

  • Large user base, with an active community and continuing feature development.
  • Simpler setup
  • Less bugs, and more community support for any bugs that we may find
  • Additional tools specifically for WordPress development

In addition, by adopting standard tools widely used in the community, we grow our ability to attract new developers to contribute to our projects.

This would fix issues #41 & #47, by removing the functionality they address in favor of using VVV.

Related

INN/deploy-tools#48

@rclations
Copy link
Contributor Author

rclations commented Aug 10, 2016


## Setup a Largo Blueprint for VV

One useful feature of VV is that it allows you to create "blueprints" for site configurations. Below, we'll setup a basic blueprint for Largo sites.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Will we need to create new blueprints for each of the Largo umbrellas?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I don't believe so - the general vv create command will walk through site setup options, where you can specify a mysql database and external repo to use for wp-content, which I think is probably the way to go.

We'll need to modify our bitbucket repo a bit so things match up better, but that should be about it.

@rclations
Copy link
Contributor Author

I think this is just about ready for review - the VVV pieces should be all set, there's just a few other related things I noticed

  1. I have removing the Vagrantfile as a step in here, but if we want to just remove that from bitbucket when we're ready, that would be cleaner
  2. using fabric to download the mysql database doesn't appear to work (at least in my testing). @benlk sent me some instructions for using curl to do this, and we should probably make that the standard.
  3. The search/replace script is a good one, but also fails for me because of the database size. Running a terminal command on the downloaded mysql file before importing it may be a better way to go. I was thinking something like sed -n 's/largoproject.wpengine.com/largo-umbrella.dev/g' mysql-edited.sql, but it didn't work on my first pass, so I want to debug that and get something really dependable setup there.

Run the following commands to initialize a new git repository and sync it with the largo-umbrella remote. This is similar to cloning a git repository, but lets us work with the files and folders we already have setup.
```
git init
git remote add origin git@bitbucket.org:rclations/largo-umbrella.git
Copy link
Collaborator

Choose a reason for hiding this comment

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

This repo is not currently public; will we be making it public or merging it into the largoproject version?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

good catch - just updated with f407bcb

5. You may then see the message "Already Installed" because you already loaded the WordPress database in a previous step.

## 13. Changing largoproject.wpengine.com to vagrant.dev.
## 12. Changing largoproject.wpengine.com to vagrant.dev.
Copy link
Collaborator

@benlk benlk Aug 25, 2016

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yeah, definitely - I'll add that in

Copy link
Contributor Author

Choose a reason for hiding this comment

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

updated in c9c02e3

------------ | -------------
Name of new site directory: | largo-umbrella
Blueprint to use (leave blank for none or use largo): | *hit [Enter]*
Blueprint to use (leave blank for none or use largo): | *hit [Enter]*
Copy link
Collaborator

Choose a reason for hiding this comment

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

doubled line here

@aschweigert aschweigert merged commit f2d2502 into INN:master Oct 4, 2016
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.

3 participants