Closed
Bug 1103045
Opened 10 years ago
Closed 10 years ago
[heartbeat] hb survey view
Categories
(Input Graveyard :: Backend, defect, P3)
Input Graveyard
Backend
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: willkg, Assigned: lgp171188, Mentored)
References
Details
(Whiteboard: u=gregg c=heartbeat p= s=input.2015q1)
We need an HB view for surveys like the addproducts view. This way, analyzers can create new surveys, enable/disable surveys and see which survey names are valid.
Reporter | ||
Updated•10 years ago
|
Assignee: nobody → willkg
Status: NEW → ASSIGNED
Reporter | ||
Comment 1•10 years ago
|
||
The totally basic immediate need of viewing surveys is in PR: https://github.com/mozilla/fjord/pull/392
Reporter | ||
Comment 2•10 years ago
|
||
Reporter | ||
Comment 3•10 years ago
|
||
That only shows the existing surveys. I didn't go the whole 9 yards to allow for creating new surveys and editing them. I'm pushing that work off until later when things aren't so crazy.
Whiteboard: u=gregg c=heartbeat p= s=input.2014q4 → u=gregg c=heartbeat p=2 s=input.2014q4
Reporter | ||
Comment 4•10 years ago
|
||
Pushed the basic view to production just now.
Will keep this open to finish up the work later.
Reporter | ||
Comment 5•10 years ago
|
||
Sliding this into Adam's pile.
Whiteboard: u=gregg c=heartbeat p=2 s=input.2014q4 → u=gregg c=heartbeat p= s=input.adam
Reporter | ||
Comment 6•10 years ago
|
||
Oopos--unassigning myself.
Assignee: willkg → nobody
Status: ASSIGNED → NEW
Reporter | ||
Comment 7•10 years ago
|
||
Tossing this in the 2015q1 queue.
Whiteboard: u=gregg c=heartbeat p= s=input.adam → u=gregg c=heartbeat p= s=input.2015q1
Reporter | ||
Comment 8•10 years ago
|
||
Bumping this to P3 since it's not super important to get done.
Priority: P1 → P3
Reporter | ||
Updated•10 years ago
|
Summary: hb survey view → [heartbeat] hb survey view
Reporter | ||
Comment 9•10 years ago
|
||
Talked to lguruprasad about this on IRC. Assigning it to him.
The summary of our conversation goes like this:
* I threw the existing heartbeat survey view together just so we had something Gregg could use, but it's missing a lot of stuff.
* I don't have a preference of sticking with function based views or switching to class based views--whichever works best is fine by me.
* The code should remain part of the analytics app since that's where all the other view code for Input for analyzers lives. That might seem a little odd, but it makes some things easier to deal with.
* The view needs to allow members of the analyzers group the ability to view, create and update heartbeat surveys.
* The users for this consist of Gregg and me and whichever other User Advocacy people end up getting involved with heartbeat. Having said that, we have a need for similar things in other areas of Input and if we had a good system for this, we could use that to build/improve the other things. For example, product maintenance.
Assignee: nobody → lgp171188
Mentor: willkg
Status: NEW → ASSIGNED
Reporter | ||
Comment 10•10 years ago
|
||
In a PR: https://github.com/mozilla/fjord/pull/490
Landed in master: https://github.com/mozilla/fjord/commit/73eaaf28d7c562e3ff0ee3aadf88b8a71f722294
That covers the "create survey" form and also reworks it to use a CBV.
I'll push it to production later.
The last bit that needs to be implemented is the ability to update/change existing surveys.
Reporter | ||
Comment 11•10 years ago
|
||
Pushed the "create survey" bits to prod just now.
Reporter | ||
Comment 12•10 years ago
|
||
Update survey form PR: https://github.com/mozilla/fjord/pull/504
Landed in master: https://github.com/mozilla/fjord/commit/3668585a294dfe960117008bbb68f76d297dd059
I'll push that out in the next deploy.
Reporter | ||
Comment 13•10 years ago
|
||
Pushed it to production just now. Marking as FIXED.
Thank you!
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Updated•8 years ago
|
Product: Input → Input Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•