Closed
Bug 1268717
Opened 9 years ago
Closed 9 years ago
Add Service Now Info to AirMo Event Record
Categories
(Webtools Graveyard :: Air Mozilla, defect)
Webtools Graveyard
Air Mozilla
Tracking
(firefox49 affected)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox49 | --- | affected |
People
(Reporter: richard, Assigned: peterbe)
Details
Attachments
(1 file)
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.
Reporter | ||
Comment 1•9 years ago
|
||
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.
Assignee | ||
Comment 2•9 years ago
|
||
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.
Reporter | ||
Comment 3•9 years ago
|
||
I like the idea. How hard is it to get the Postgres server updated?
Assignee | ||
Comment 4•9 years ago
|
||
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.
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → peterbe
Assignee | ||
Comment 5•9 years ago
|
||
Comment 6•9 years ago
|
||
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)
Updated•9 years ago
|
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Updated•4 years ago
|
Product: Webtools → Webtools Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•