1268717 - Add Service Now Info to AirMo Event Record

Status: RESOLVED FIXED

(Webtools Graveyard :: Air Mozilla, defect)

Description

[Richard A Milewski[:richard]]

It would be helpful to add two fields to each Air Mozilla Event Record.  Laura would like to add these at the time the event request is approved.  (i.e. entered on the event submission approval screen).

They should not be required fields.  They should show on the event edit page.

Comment 1

[Richard A Milewski[:richard]]

The two fields are a Service Now Ticket number and a link to that ticket in Service Now.   If we can figure out how to deduce the url from the ticket number then we only have to enter the ticket number and we can synthesize the link.

Comment 2

[Peter Bengtsson [:peterbe]]

How about instead each event has a little JSON field with key-value pairs. The advantage is that we can add more key-values to the event without having to make new database migrations. Just this morning, I thought of another field I would like add; language. 

In the event manager edit page we could have a little table of Keys and Values that you can add/edit/remove from. 

Note, you won't easily be able to query based on this. E.g We can't easily get a list of events whose KEY is equal to VALUE. Note2: If we upgrade our Postgres server this would be possible using the new JSON type in Postgres.

Comment 3

[Richard A Milewski[:richard]]

I like the idea.  How hard is it to get the Postgres server updated?

Comment 4

[Peter Bengtsson [:peterbe]]

If we upgrade our PG to 9.4 (we're on 9.2 right now) AND we upgrade to Django 1.9 THEN we'd be able to use the native Postgres JSONField.
See https://bugzilla.mozilla.org/show_bug.cgi?id=1124348

However, we can start gently by faking the JSON field. It means it will work but things will be slow when it comes to doing queries on keys/values within.

Comment 5

[Peter Bengtsson [:peterbe]]

Attachment: Link to Github pull-request: https://github.com/mozilla/airmozilla/pull/744

Comment 6

[[github robot]]

Commit pushed to master at https://github.com/mozilla/airmozilla

https://github.com/mozilla/airmozilla/commit/33bb654f167ab14e7301995ed10e81ae024232fd
fixes bug 1268717 - misc key/value store for events (#744)