Add room number field for events
Where should it go? If it's part of Venue, then we'll have different venues for each room number. So, presumably, separate.
Assigning to Sam for suggestions.
#1 Updated by Samantha Raddatz almost 4 years ago
Yes, as long as we keep the auto-fill/dropdown for the building addresses it should be separate. Such as:
Venue: (auto-fill/dropdown for building address)
Room: (open-text field)
See attached for a proposed layout.
I originally thought the text field should be titled "Room number", but an event could take place in an auditorium or other space that doesn't have a room number assigned to it.
#3 Updated by Samantha Raddatz almost 4 years ago
Currently, all of the fields are technically optional, so I don't think there's any need to state that these are.
We may want to consider making some fields required so that people aren't posting blank or partially-entered events (though blank events that are published don't appear in the calendar), in which case anything optional should be labeled as such.
#8 Updated by Boone Gorges almost 3 years ago
- Status changed from Assigned to Testing Required
This is ready to test on cdev. See http://cdev.gc.cuny.edu/members/boonebgorges/events/kegger/
The relevant changeset is https://github.com/cuny-academic-commons/bp-event-organiser/commit/35adce69311a6fc0011193eb2138daccaa4f73d3, though I had to make a hotfix to event-organiser to make it work on CAC. I've opened a PR so that this hotfix can be deprecated with the next version of EO. https://github.com/stephenharris/Event-Organiser/pull/350
#9 Updated by Samantha Raddatz almost 3 years ago
Thanks, Boone! I tested it out and the room functionality works great.
The map was acting weird though -- when I chose the Graduate Center from the dropdown it dropped a pin in some random place in the middle of an ocean (screenshot attached). Not sure if this is a cdev bug or related to the changes you made, but I figure it's of note.