Project

General

Profile

Bug #29295

Change string to number in API docs for ids

Added by Oleh Fedorenko over 1 year ago. Updated 3 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Adi Abramovitch
Category:
API
Target version:
-
Difficulty:
easy
Triaged:
No
Bugzilla link:
Pull request:
https://github.com/theforeman/foreman/pull/8400, https://github.com/theforeman/foreman/pull/7863
Fixed in Releases:
2.5.0
Found in Releases:

Description

https://community.theforeman.org/t/unable-to-create-or-update-managed-host-via-api/17692 reports that operatingsystem_id should be a nubmer instead of a string in the API documentation. So, the main reason for this issue is to fix this entry.

However, I've noticed that we've got similar inconsistency for ID entries in more than one API doc controller. Those should be probably reviewed to prevent similar issues in the future.

Associated revisions

Revision 39708d2c (diff)
Added by Jaskaran Singh Narula 9 months ago

Refs #29295 - fixes the API doc

Revision 04ac0de6 (diff)
Added by Adi Abramovitch 3 months ago

Fixes #29295 - Change operatingsystem_id type from string to number in API docs for ids

History

#1 Updated by The Foreman Bot 11 months ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/7863 added

#2 Updated by Tomer Brisker 11 months ago

  • Category set to API

#3 Updated by Adi Abramovitch 3 months ago

  • Assignee set to Adi Abramovitch

#4 Updated by The Foreman Bot 3 months ago

  • Pull request https://github.com/theforeman/foreman/pull/8400 added

#5 Updated by The Foreman Bot 3 months ago

  • Fixed in Releases 2.5.0 added

#6 Updated by Adi Abramovitch 3 months ago

  • Status changed from Ready For Testing to Closed

Also available in: Atom PDF