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

Update Tshirt size to all sizes #105

Open
bendygirl opened this issue May 30, 2015 · 4 comments
Open

Update Tshirt size to all sizes #105

bendygirl opened this issue May 30, 2015 · 4 comments
Assignees
Labels

Comments

@bendygirl
Copy link

We will need to update the fields AND email all attendees to update this information. Tim, I can do this, but it won't be in the feature, so do you want to do this in dev and test first or throw caution to the wind and do it on prod?

@timwood
Copy link
Contributor

timwood commented Jun 7, 2015

@bendygirl The defaults for COD are broken up by Mens and Womens XS - XL. We added our own values of Small, Large and 2XL. Can you define "all sizes" so I know what to do about this? If we go back to Men/Women, I'll have to make a decision about how to manipulate (update hook) the existing data into the COD defaults (Men's vs. Women's) or just leave our values and we can make an assumption about the existing sizes being Men's and mention that when we notify registrants.

@timwood
Copy link
Contributor

timwood commented Jun 7, 2015

I'd prefer not to have to write an update hook.

@timwood
Copy link
Contributor

timwood commented Jun 10, 2015

@bendygirl Any thoughts on this? We didn't do anything with this over the sprint. Is the t-shirt printing date looming?

@bendygirl
Copy link
Author

we can leave it for now.

On Wed, Jun 10, 2015 at 4:26 PM, Tim Wood [email protected] wrote:

@bendygirl https://github.com/bendygirl Any thoughts on this? We didn't
do anything with this over the sprint. Is the t-shirt printing date looming?


Reply to this email directly or view it on GitHub
#105 (comment)
.

Kirsten
http://uniongal.blogspot.com
Join Uniongal on Reddit http://www.reddit.com/r/Uniongal/

@timwood timwood removed the SPRINT label Jun 10, 2015
@timwood timwood removed this from the send to manufacturer milestone Jun 16, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants