Project

General

Profile

Actions
Copy link

Task #2845

closed

Review branch 2525-java-sdk

Added by Radhika Chippada almost 10 years ago. Updated almost 10 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
-
Target version:
2014-05-28 Pipeline Factory
Actions
Copy link
 #1

Updated by Radhika Chippada almost 10 years ago

Minor updates:
- Split the list of parameters returned for a method into required and optional lists to enhance usability.
- Fixed array to json conversion conversion in putParameter method. Previously, this was working only for array of array (filter) case and hence corrected to work with just array objects as well.

Actions
Copy link
 #2

Updated by Peter Amstutz almost 10 years ago

  1. returning getAvailableParametersForMethod() as a map with just "required" and "optional" seems awkward, how about returning it as an array with two items?
  2. getJsonValueFromListType() uses Arrays.deepToString. Is this guaranteed to produce valid JSON?
Actions
Copy link
 #3

Updated by Radhika Chippada almost 10 years ago

  • Status changed from New to Resolved
  • Start date set to 05/30/2014
  • Remaining (hours) set to 0.0

addressed this feedback.

Actions
Copy link

Also available in: Atom PDF