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

Add a name field to work_sites #107

Closed
glroman opened this issue Sep 26, 2016 · 8 comments
Closed

Add a name field to work_sites #107

glroman opened this issue Sep 26, 2016 · 8 comments

Comments

@glroman
Copy link
Contributor

glroman commented Sep 26, 2016

Add a name field to the work_sites table which can be used when displaying and selecting work_sites. On work_site creation, the name field should be required and unique. The address field should remain required and be shown when viewing work_site details.

@glroman
Copy link
Contributor Author

glroman commented Sep 26, 2016

I'll tackle this; thanks. It's blocking #55.

@bjmllr
Copy link
Member

bjmllr commented Sep 26, 2016

@glazomaniac Do work sites have names?

@glroman
Copy link
Contributor Author

glroman commented Sep 26, 2016

Sorry, I didn't realize the question was still outstanding. From reading the material on the http://www.habitat-nola.org site, I saw these and thought they must have names for the various projects:

  • Musicians’ Village
  • Faith Build
  • Jimmy and Rosalynn Carter Work Project
  • Blitz Build
  • America Street

I can try finishing #55 without this and then change it to accommodate names if/when we hear back that it's OK.

@bjmllr
Copy link
Member

bjmllr commented Sep 26, 2016

Looking at the database, right now we have 3 separate work sites on America Street. You can see them all at https://habitat-nola-signin.herokuapp.com/ by checking the work site dropdown.

@glroman
Copy link
Contributor Author

glroman commented Sep 26, 2016

Got it. So it's not looking good for the name idea. Wonder if they'd want to group things by project? If so, then something like "America Street, WS001" might be useful. The problem I see now is that if their active list ever gets to be large, the drop-downs get unwieldy. And over time, active + inactive gets large and the only application key into that list is the address.

@glazomaniac
Copy link
Collaborator

We sometimes have names for special builds or projects, but yeah, it's better if we just do this all by address. Thanks for the thought, though!

@glazomaniac
Copy link
Collaborator

We'll never be building more than 10-20 addresses at a time--our staff just isn't big enough for that. Usually it's around 10 or 12. So it shouldn't be a problem regarding the dropdowns if the inactive sites are able to be removed from them when we're done.

@glroman
Copy link
Contributor Author

glroman commented Sep 27, 2016

@glazomaniac Thanks for the feedback; I'm closing this issue.

@glroman glroman closed this as completed Sep 27, 2016
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

No branches or pull requests

3 participants