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

Fixing bootstrap process with default values. #31

Closed
wants to merge 2 commits into from

Conversation

sjlu
Copy link

@sjlu sjlu commented Nov 18, 2013

Errored out when it asked for these columns.

@snipe
Copy link
Owner

snipe commented Nov 18, 2013

Steps to reproduce? Not seeing that error here.

@sjlu
Copy link
Author

sjlu commented Nov 18, 2013

Just ran php artisan app:install and received the following error.

http://d.pr/i/mIZ6

@snipe
Copy link
Owner

snipe commented Nov 18, 2013

I think you installed while I was in the middle of breaking something. This code shouldn't be necessary. Hang tight - fixing something else I broke, but will get back to this.

I assume you ran the migration as well?

@sjlu
Copy link
Author

sjlu commented Nov 18, 2013

Yeah, those migrations were run. I had a feeling you probably had something
in your local copy.

On Monday, November 18, 2013, snipe wrote:

I think you installed while I was in the middle of breaking something.
This code shouldn't be necessary. Hang tight - fixing something else I
broke, but will get back to this.

I assume you ran the migration as well?


Reply to this email directly or view it on GitHubhttps://github.com//pull/31#issuecomment-28710585
.

@snipe
Copy link
Owner

snipe commented Nov 18, 2013

Yeah, I don't want to ask people all of these questions on setup. Too much crap to enter, when it's not really all that important just to get started.

@snipe
Copy link
Owner

snipe commented Nov 18, 2013

Can you wipe it and try again? ad914b7 should have fixed this.

@snipe
Copy link
Owner

snipe commented Nov 18, 2013

Sorry, I meant c679d44

@snipe
Copy link
Owner

snipe commented Nov 18, 2013

Confirmed this solved worked on a fresh install. Closing this pull request. In other words....

DENIED.

👅

@snipe snipe closed this Nov 18, 2013
@sjlu
Copy link
Author

sjlu commented Nov 19, 2013

lol.

@kwinters1 kwinters1 mentioned this pull request Apr 16, 2015
@cbinstead cbinstead mentioned this pull request Jun 9, 2015
@craig7716 craig7716 mentioned this pull request Jun 12, 2015
@bailantilles bailantilles mentioned this pull request Mar 30, 2016
2 tasks
@zmorrow zmorrow mentioned this pull request May 20, 2016
2 tasks
@cookierun22 cookierun22 mentioned this pull request Jan 28, 2023
2 tasks
@Bjufen Bjufen mentioned this pull request Feb 9, 2023
2 tasks
This was referenced Aug 22, 2023
@jayavman jayavman mentioned this pull request Sep 2, 2023
2 tasks
@manhvt20 manhvt20 mentioned this pull request Mar 18, 2024
2 tasks
@uglycrab uglycrab mentioned this pull request Mar 27, 2024
2 tasks
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.

None yet

2 participants