Thanks for all your work on this! As I document, I noticed a few minor text issues:
"Email" should be "e-mail" (label and text)
"The address you select here will be when" s/b
"The address you select here will be used when"
"git" s/b "Git" (two occurrences)
If I get enough time on my hands, I will try to submit a patch, but that's not my skill set so much.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | cvsmigration-1018658-2.patch | 7.72 KB | jonathan webb |
| #2 | cvsmigration-1018658.patch | 9.98 KB | jonathan webb |
Comments
Comment #1
jonathan webb commentedI can handle the patch for this.
Comment #2
jonathan webb commentedPatch ready for review. I believe I caught all of the "email" spellings. Just note that there are some mods to the interface in #1013740: Adding a "Full Name" Preference Field, I resubmitted my patch for that with similar corrections to what you have here.
Comment #3
jonathan webb commentedUpdated patch to work with latest HEAD.
Comment #4
sdboyer commentedLooks good to me. Go ahead and commit this thing...we need a stable release of cvsmigration soon here, so that we can get it deployed on d.o. :)
Comment #5
jonathan webb commentedPatch committed!
Comment #6
jonathan webb commentedPatch committed!