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

Meetings #1450

Draft
wants to merge 4 commits into
base: master
from

Conversation

Projects
None yet
1 participant
@tiltec
Copy link
Member

commented Apr 24, 2019

Backend here: yunity/karrot-backend#710

  • new sidebar entry for group meetings
  • no page yet for meetings in a place, waiting for #1432 to be merged
  • added meeting create to "pickup manage" as a new section - only one-time meetings for now
  • added a event datastore generator, the meeting and pickup datastores extend it
  • meeting API file is copied from pickup API, could be a generator as well

Observations/Questions:

  • a lot of code for something that basically already exists...
  • there should be a better way of managing events than a long list of collapsibles. might become confusing if we add more types
  • should we show the place meetings page even if there are no meetings in a place?
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.