Project

General

Profile

Actions

Bug #7372

closed

API v2 - accept PUT/POST requests with wrapped root node to add/remove has_many associations of child nodes

Added by Joseph Magen over 9 years ago. Updated almost 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
API
Target version:
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

Description

previous commit bbf64d9900262e42afaec6dd84934ae601c1d4f3 only checked for has_many associations of child nodes if the request was not wrapped (no root node). Since POST/PUT is documented as wrapped, it should support this to.
Therefore, there will be two mains to add/remove assocations, either by

1) passing *_ids

{
"operatingsystem": {
"name": "CentOs",
"config_template_ids": [36, 12, 40]
}
}

2) Passing child nodes

{
"operatingsystem": {
"name": "CentOs",
"config_templates": [ {
"id": 36,
"name": "CentOS-enabled-post"
}, {
"id": 12,
"name": "Grubby Default"
}, {
"id": 40,
"name": "Katello Kickstart Default",
}
]
}
}

Actions #1

Updated by The Foreman Bot over 9 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/smart-proxy/pull/211 added
  • Pull request deleted ()
Actions #2

Updated by Dominic Cleal over 9 years ago

  • Status changed from Ready For Testing to Assigned
  • Pull request added
  • Pull request deleted (https://github.com/theforeman/smart-proxy/pull/211)
Actions #3

Updated by The Foreman Bot over 9 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/smart-proxy/pull/211 added
  • Pull request deleted ()
Actions #4

Updated by Dominic Cleal over 9 years ago

  • Status changed from Ready For Testing to Assigned
  • Target version deleted (1.7.4)
  • Pull request added
  • Pull request deleted (https://github.com/theforeman/smart-proxy/pull/211)
Actions #5

Updated by Joseph Magen over 9 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/1745 added
  • Pull request deleted ()
Actions #6

Updated by The Foreman Bot over 9 years ago

  • Target version set to 1.7.2
Actions #7

Updated by Joseph Magen over 9 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100
Actions #8

Updated by Dominic Cleal over 9 years ago

  • translation missing: en.field_release set to 21
Actions

Also available in: Atom PDF