SiT! Bugs - SiT!
View Issue Details
0000945SiT!feedbackpublic2009-09-16 21:022011-06-17 14:13
3.90beta1Current GIT 
0000945: Feedbacks should be enabled pr site.
If/When a site gets its own Control panel.

Feedback forms should be added in the site profile.

This gives the power to direct the feedback forms to a dedicated site
instead of using a "Generic" all-around form.

But also keeping the settings for feedback from all using the Generic form.
No tags attached.
related to 0001176confirmed  Contacts should have their own profile settings page 
child of 0000946confirmed  Sites should have their own profile settings page 
child of 0001458resolved Tomse [META] Working feedback system 
Issue History
2009-09-16 21:02TomseNew Issue
2009-09-17 13:28ivanNote Added: 0001970
2009-09-17 13:28ivanStatusnew => feedback
2009-09-17 13:45nicdevNote Added: 0001971
2009-09-17 15:14TomseNote Added: 0001975
2010-03-21 19:00paulhNote Added: 0002809
2010-03-21 22:40TomseNote Added: 0002820
2010-03-21 22:40TomseStatusfeedback => new
2010-03-22 07:51nicdevNote Added: 0002821
2010-03-22 08:47ivanNote Added: 0002822
2010-03-22 10:08ivanNote Added: 0002823
2010-03-22 10:38nicdevNote Added: 0002824
2010-03-22 10:52ivanNote Added: 0002825
2010-03-22 12:41nicdevNote Added: 0002826
2010-03-22 12:42nicdevNote Edited: 0002826bug_revision_view_page.php?bugnote_id=2826#r114
2010-03-22 13:17ivanNote Added: 0002829
2010-03-22 13:17ivanProduct Version4.0 =>
2010-03-22 13:17ivanAssigned To => ivan
2010-03-22 13:17ivanStatusnew => confirmed
2010-03-22 13:17ivanAssigned Toivan =>
2010-03-22 13:27ivanRelationship addedchild of 0000946
2010-03-22 13:31ivanRelationship addedrelated to 0001176
2010-03-24 20:38ivanTarget Version => 4.1
2011-02-14 13:54ivanTarget Version4.1 => 3.90beta1
2011-02-14 14:02ivanRelationship addedchild of 0001458
2011-02-16 17:00TomseAssigned To => Tomse
2011-02-16 17:00TomseStatusconfirmed => assigned
2011-06-17 10:52TomseStatusassigned => confirmed
2011-06-17 14:13TomseNote Added: 0004031
2011-06-17 14:13TomseStatusconfirmed => resolved
2011-06-17 14:13TomseFixed in Version => Current GIT
2011-06-17 14:13TomseResolutionopen => fixed

2009-09-17 13:28   
I'm not sure what you mean by this exactly. Do you mean to have a specific feedback form for a specific site? Like a form made just for one customer?

I don't know if you're aware, because it's not documented... I'll fix that right after posting this. But you can prevent certain sites from receiving feedback forms completely by going to SiT! | Control Panel | Features | Feedback and setting "No-Feedback Contracts". That sounds at least semi-related to this.
2009-09-17 13:45   
Hi Ivan,

Just to add my 3 cents worth ;-)

We have been looking at using the feedback as well, and from our perspective, it would be handy to have a "tickbox" much like the "data protection stuff" that says "do not send feedback to this customer".

I know we can exclude contracts in the config file but it becomes a bit hairy in if you have a lot of contracts, and it is difficult to keep track as they are numbered, and not names ..

Preferably we would want to limit the feedback by "contact" and not just "contract" as we have a lot of contracts where the manager is a contact and he may not be one of the people that wants to receive the feedback form..

Hope it helps!
2009-09-17 15:14   
you got it right in the first part.
to target a specific customer with a specific feedback form.

consider this example
3 customers
Navy, Air force and Army

q: so how is it to be a sailor ?
2010-03-21 19:00   
Hi Tomse,

Is this one still relevent? Feedback is relating to the incident so I'd have thought the questions would be the same? From your suggestion it seems your suggesting SiT becomes a surveying tool? I could of course be getting the wrong end of the stick
2010-03-21 22:40   
I'm not sure if it is at all necessary, but it was an idea I thought needed to come out.

I guess that there isn't a need for customizable feedbacks targeting individual customers. :-)
2010-03-22 07:51   
Hi Ivan,

Just a note. I misunderstood what tomse meant in a way, but the title of this request is a valid one in the end. It would be good to have a selection in the customer/Site config where you can say "exclude from feedback" like the "data protection" bit in the contact's settings.

This makes it easier to manage when you have 288 sites and want to send feedback only to "certain contacts".

2010-03-22 08:47   
There is already a way to exclude a site from feedback via system configuration, but yes an easier way would be nice, this is something we could do once we have per-site and per-contact configuration settings.
2010-03-22 10:08   
Well I was slightly wrong, in fact there's a way to exclude /contacts/ from feedback but not whole sites. see [^]
2010-03-22 10:38   

I see what you mean ... by excluding a contact's contract in the list, you exclude the contact himself. The problem there is that connected to a certain contract we may have many contacts, and we may just want to exclude some contacts not all for that contract ... does that make sense?

We were discussing this morning about this with Management, and i suppose it depends on each installation of SiT ... for us we would like to see the possibility to exclude/include "contacts" individually from the feedback, in effect to be able to narrow it down to individuals ...

Even if this has no GUI in the beginning, and is the same as the list used to exclude "contracts" it will be extremely useful ;)
2010-03-22 10:52   
I definitely think it would be useful, in fact we might want to give the contact the ability to turn off feedback him/herself, just as he can do with data protection settings in the portal. If a customer doesn't want to fill in feedback forms trying to force him/her to do so is rather pointless.
2010-03-22 12:41   
(edited on: 2010-03-22 12:42)
I was playing around with this earlier and i have a workaround until 4.x is around...

I created a new function "contact_feedback_locked($contactid)" which checks a new config array "$CONFIG['contact_feedback_disabled']=array()", and returns TRUE when the contact is in the array list.
Then in line 489 of "incident_close.php" we add the following:

++ $contact_blocked = contact_feedback_locked((incident_contact($id));

and add another "AND" to the
if ($CONFIG['feedback_form'] != '' AND $CONFIG['feedback_form'] > 0 AND $send_feedback == TRUE AND $contact_blocked ==TRUE)

This is a crude way, just as the exclusion of contracts but it works(tested it).

EDIT: BTW Ivan i like your idea of making it possible for the user to turn it off.

2010-03-22 13:17   
Confirming this, but I don't think we'll manage it for 4.0 so removing that target. It needs contact settings first.
2011-06-17 14:13   
fixed in : fcbf9a2