1026677 - Add Attendees table & model to events-service

Status: RESOLVED FIXED

(Webmaker Graveyard :: Events, defect)

Description

[Jon Buckley [:jbuck]]

We're adding an Attendees table that both the rsvp and mentor/co-organizers feature will share.

It'll be kinda like:

INTEGER event_id
INTEGER user_id
BOOLEAN didRSVP
BOOLEAN didAttend
BOOLEAN isMentor
BOOLEAN isCoorganizer (better name would be awesome)

Comment 1

[Gavin Suntop [:gvn]]

https://etherpad.mozilla.org/tRokPYUG9U

Comment 2

[Gavin Suntop [:gvn]]

Attachment: https://github.com/mozilla/webmaker-events-service/pull/27

Note that there's a new field in the Event model too.

Comment 3

[Jon Buckley [:jbuck]]

Comment on attachment 8441743 [details] [review]
https://github.com/mozilla/webmaker-events-service/pull/27

r- with notes in the PR.

:gvn - Could you create a separate PR with just the content of `models/`? That stuff is nearly ready to go, so we can land that now, and work through the rest of the controllers more easily

Comment 4

[Gavin Suntop [:gvn]]

New models:
https://github.com/mozilla/webmaker-events-service/pull/28

Comment 5

[Gavin Suntop [:gvn]]

Attachment: https://github.com/mozilla/webmaker-events-service/pull/28

Comment 6

[Chris DeCairos (:cade)]

Comment on attachment 8441743 [details] [review]
https://github.com/mozilla/webmaker-events-service/pull/27

Overall this looks good, but I was able to rsvp for events that don't exist yet:

repos  curl -XPOST 'http://localhost:1989/rsvp/?eventid=21&userid=3' -H 'Cookie: <cookie>'

Where event 21 isn't a real event.

Comment 7

[Chris DeCairos (:cade)]

Comment on attachment 8441743 [details] [review]
https://github.com/mozilla/webmaker-events-service/pull/27

well that was confusing.

Comment 8

[Gavin Suntop [:gvn]]

Comment on attachment 8441743 [details] [review]
https://github.com/mozilla/webmaker-events-service/pull/27

Ah, good catch! Should be fixed now...

Comment 9

[Gavin Suntop [:gvn]]

Landed on master. Not yet deployed.

Comment 10

[Gavin Suntop [:gvn]]

Jbuck, can you do the DB schema migrations? (Or maybe you already did?)

Comment 11

[Jon Buckley [:jbuck]]

All set on staging, waiting on prod until next week.