Project

General

Profile

Bug #29295

Change string to number in API docs for ids

Added by Oleh Fedorenko over 2 years ago. Updated over 1 year 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 almost 2 years ago

Refs #29295 - fixes the API doc

Revision 04ac0de6 (diff)
Added by Adi Abramovitch over 1 year ago

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

History

#1 Updated by The Foreman Bot about 2 years ago

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

#2 Updated by Tomer Brisker about 2 years ago

  • Category set to API

#3 Updated by Adi Abramovitch over 1 year ago

  • Assignee set to Adi Abramovitch

#4 Updated by The Foreman Bot over 1 year ago

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

#5 Updated by The Foreman Bot over 1 year ago

  • Fixed in Releases 2.5.0 added

#6 Updated by Adi Abramovitch over 1 year ago

  • Status changed from Ready For Testing to Closed

Also available in: Atom PDF