search

ac3cloud / roust

Ruby client for Request Tracker's REST API
Other
6 stars 4 forks source link

Add support for adding and removing ticket links #14

Closed auxesis closed 9 years ago

auxesis commented 9 years ago

Adds 2 methods:

These are pretty self descriptive.

Fixes #7.

Sarah-E-Greene commented 9 years ago

Were you planning to extend ticket creation/update to call this method if links are given?

Sarah-E-Greene commented 9 years ago

The requestors / ccs / admin CCs allow a comma separated string, as well as a hash. For consistency, it would be nice to support both here.

I also note that for each refersto link that is set or deleted, an email is sent out - so if there are 3 links already, and you add one more, RT sends out 7 emails (one for each of the deletes (3), and one for each of the adds (4)). This could be a configuration issue in our RT instance.

jamieslater commented 9 years ago

This is a bug in RT

From: Sarah Kowalik [mailto:notifications@github.com] Sent: Tuesday, 7 October 2014 1:31 PM To: bulletproofnetworks/roust Cc: Jamie Slater Subject: Re: [roust] Add support for adding and removing ticket links (#14)

This also stops working if you delete the links, and then try to re-add the same links in, with no apparent error message or change to RT. Of course, that could be a bug in RT, not roust.

— Reply to this email directly or view it on GitHubhttps://github.com/bulletproofnetworks/roust/pull/14#issuecomment-58128584.

jamieslater commented 9 years ago

I also note that for each refersto link that is set or deleted, an email is sent out - so if there are 3 links already, and you add one more, RT sends out 7 emails (one for each of the deletes (3), and one for each of the adds (4)). This could be a configuration issue in our RT instance.

This is a known RT issue

Sarah-E-Greene commented 9 years ago

This is a bug in RT

I think this was actually PEBKAC.