rt icon indicating copy to clipboard operation
rt copied to clipboard

REST 1.0 - Add fetching and setting of transaction CustomFields

Open puck opened this issue 8 months ago • 3 comments

These patches allow CustomFields to be set on transactions via the REST API. Thread on rt-devel is here: https://lists.bestpractical.com/pipermail/rt-devel/2016-March/012530.html

It is a resurrection of https://github.com/bestpractical/rt/pull/178 which I expect was automatically closed because the branch name started with 4.2. This patch set applies cleanly against 4.4-trunk and 5.0-trunk.

It is also the patchset from https://github.com/bestpractical/rt/pull/183 which was closed in favour of 178.

Cheers, Andrew

puck avatar Oct 20 '23 23:10 puck

We have generally been trying not to make any changes to REST 1.0 code, leaving it in place for stability and backward compatibility. REST 2 supports transaction CFs, but I assume you're adding on to something that already uses REST 1?

cbrandtbuffalo avatar Oct 23 '23 18:10 cbrandtbuffalo

Haha, understandable.

I was looking at what patches we were still carrying locally. We're no longer using the REST 1.0 API, so I was planning on dropping these patches, but recall that others have been using them, so thought I'd refresh the branch.

puck avatar Oct 24 '23 04:10 puck

OK, cool. In the interest of stability for REST 1.0 and also keeping this code available, I'll plan to leave this branch here for anyone who might need it. If the patches apply cleanly now, they should continue to do so if we leave the REST 1.0 code alone (except for any security issues).

cbrandtbuffalo avatar Oct 24 '23 13:10 cbrandtbuffalo