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.

Sign up for GitHub

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

Jump to bottom

Switch Venue Type to a dedicated flag #449

Closed
srogers opened this issue Jun 6, 2020 · 1 comment
Closed

Switch Venue Type to a dedicated flag #449

srogers opened this issue Jun 6, 2020 · 1 comment
Labels
enhancement New feature or request Fixed on staging

Comments

@srogers
Copy link
Owner

srogers commented Jun 6, 2020 •
edited
Loading

Rather than hacking virtual/multiple into the venue field as a name. This is weird if it's set and then unset.

Setting "Multiple" or "Virtual" should clear out the location fields entirely so cities and states don't get mis-counted in aggregate data. Probably should clean it in the model, so expanding/collapsing is not destructive (particularly not in update).
@srogers srogers added the enhancement New feature or request label Jun 6, 2020
@srogers srogers changed the title Switch Venue "Multiple" to a dedicated flag Switch Venue Type to a dedicated flag Aug 3, 2020
srogers added a commit that referenced this issue Aug 3, 2020
@srogers
#449 Switch venue / presentation type to a dedicated attribute
81ee073
srogers added a commit that referenced this issue Aug 3, 2020
@srogers
#449 don't offer "Multiple" as a choice for presentation location typ…
8f705b0 
…e - only makes sense for events.
@srogers
Copy link
Owner Author

srogers commented Sep 18, 2020

Deployed

@srogers srogers closed this as completed Sep 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request Fixed on staging
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@srogers