Skip to content

Username changes#13

Merged
6 commits merged intosdboyer:import-all-stagingfrom
JoshOldenburg:import-all-staging
Jan 31, 2011
Merged

Username changes#13
6 commits merged intosdboyer:import-all-stagingfrom
JoshOldenburg:import-all-staging

Conversation

@JoshOldenburg
Copy link
Copy Markdown

@sdboyer
Copy link
Copy Markdown
Owner

sdboyer commented Jan 25, 2011

Hmm...I might want to loosen that restriction, though...I'll ponder that.

@JoshOldenburg
Copy link
Copy Markdown
Author

We should at least let the . Are we verifying that anywhere? I don't see it in the script.

@sdboyer
Copy link
Copy Markdown
Owner

sdboyer commented Jan 26, 2011

Verification is in the form field. The CVS character set is even more restrictive - alphanumerics ONLY - so we don't have to validate that existing data.

@sdboyer
Copy link
Copy Markdown
Owner

sdboyer commented Jan 26, 2011

ugh i don't like how they do their buttons, i keep hitting comment & close

@JoshOldenburg
Copy link
Copy Markdown
Author

:) Yikes: I could have been JoshTheGeek in the first place!

@sdboyer
Copy link
Copy Markdown
Owner

sdboyer commented Jan 27, 2011

Actually, CVS was restricted to non-capital alphanumerics. I think.

@JoshOldenburg
Copy link
Copy Markdown
Author

Makes sense: bad software has very bad restrictions

@sdboyer
Copy link
Copy Markdown
Owner

sdboyer commented Jan 31, 2011

No need to merge in my upstream periodically unless there's actually a conflict that has to be resolved. Just generates merge noise.

Pulling in the latest of this in preparation for public testing, but keep 'em coming.

@sdboyer
Copy link
Copy Markdown
Owner

sdboyer commented Jan 31, 2011

Please be careful...4299 wanted 'reyero', but you had 'rayero'. I only noticed because I know Jose and it didn't look right.

@sdboyer
Copy link
Copy Markdown
Owner

sdboyer commented Jan 31, 2011

This broke the build last night, three times - there was at least one user (svendecabooter) who requested a git username that violated unique key constraints.

@JoshOldenburg
Copy link
Copy Markdown
Author

Oi! Now reyero. Thank you.
How do I check if there is a duplicate username?

@sdboyer
Copy link
Copy Markdown
Owner

sdboyer commented Feb 1, 2011

Turns out svendecabooter's at fault there.

@sdboyer
Copy link
Copy Markdown
Owner

sdboyer commented Feb 2, 2011

Can I get an update on this?

@timplunkett
Copy link
Copy Markdown

241634 => 'timplunkett',
should be
241634 => 'tim.plunkett',

Thanks!

@JoshOldenburg
Copy link
Copy Markdown
Author

sdboyer: Sorry. Working on it.

…_p, add period to john, fix typo in reyero (now using copy paste), and fix tim.plunkett
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Sorry, reyero.

@JoshOldenburg
Copy link
Copy Markdown
Author

Ok. Up to date.

@sdboyer
Copy link
Copy Markdown
Owner

sdboyer commented Feb 7, 2011

Got em, using these in the next build.

@JoshOldenburg
Copy link
Copy Markdown
Author

Cool! 87 people have responded so far, not suprising

@JoshOldenburg
Copy link
Copy Markdown
Author

Closed. sdboyer, you might as well close this after you merge the rest.

@sdboyer
Copy link
Copy Markdown
Owner

sdboyer commented Feb 16, 2011

OK, we're all up to date. Thanks for shepharding this for me!

This pull request was closed.
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.

3 participants