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

Remove unneeded password_confirmation from seed. #7878

Merged
merged 1 commit into from Jan 18, 2015

Conversation

4 participants
@cirosantilli
Contributor

cirosantilli commented Sep 27, 2014

Not validated if not present: plataformatec/devise#2796

Already works like that for the user development seeds.

@TeatroIO

This comment has been minimized.

TeatroIO commented Sep 27, 2014

I've prepared a stage. Click to open.

@cirosantilli cirosantilli force-pushed the cirosantilli:rm-seed-password-confirmation branch 2 times, most recently from 5ef2687 to 996ef66 Oct 1, 2014

@jvanbaarsen

This comment has been minimized.

Contributor

jvanbaarsen commented Dec 14, 2014

@cirosantilli Can you please make sure this is mergeable?

@cirosantilli cirosantilli force-pushed the cirosantilli:rm-seed-password-confirmation branch from 996ef66 to faab452 Dec 28, 2014

@cirosantilli

This comment has been minimized.

Contributor

cirosantilli commented Dec 28, 2014

@jvanbaarsen rebased. But Semaphore does not want to run?

@cirosantilli

This comment has been minimized.

Contributor

cirosantilli commented Dec 29, 2014

OK, semaphore passed.

@jvanbaarsen jvanbaarsen added this to the 7.7 milestone Jan 4, 2015

@dzaporozhets dzaporozhets modified the milestones: 7.7, 7.8 Jan 13, 2015

dzaporozhets added a commit that referenced this pull request Jan 18, 2015

Merge pull request #7878 from cirosantilli/rm-seed-password-confirmation
Remove unneeded password_confirmation from seed.

@dzaporozhets dzaporozhets merged commit 577fe18 into gitlabhq:master Jan 18, 2015

2 checks passed

default The build passed on Semaphore.
Details
semaphoreci The build passed on Semaphore.
Details

@cirosantilli cirosantilli deleted the cirosantilli:rm-seed-password-confirmation branch Jan 18, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment