Project

General

Profile

Bug #33347

Settings search is broken in hammer

Added by Oleh Fedorenko 9 months ago. Updated 7 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Oleh Fedorenko
Category:
Settings
Target version:
3.0.1
Difficulty:
trivial
Triaged:
No
Bugzilla link:
Pull request:
https://github.com/theforeman/foreman/pull/8825, https://github.com/theforeman/foreman/pull/8742
Fixed in Releases:
3.0.1, 3.1.0
Found in Releases:

Description

There are two issues:

1. Page/per_page is not being translated to integer which leads to `undefined method '-' for "1":String`.
2. Deafult name -> id resolver is broken also since API doesn't understand "search": "name = \"query_local_nameservers\"". It worked before.

Associated revisions

Revision 2bafa941 (diff)
Added by Oleh Fedorenko 9 months ago

Fixes #33347 - Fix settings index API

History

#1 Updated by The Foreman Bot 9 months ago

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

#2 Updated by The Foreman Bot 9 months ago

  • Fixed in Releases 3.1.0 added

#3 Updated by Oleh Fedorenko 9 months ago

  • Status changed from Ready For Testing to Closed

#4 Updated by Evgeni Golov 8 months ago

  • Target version set to 3.0.1

#5 Updated by The Foreman Bot 8 months ago

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

#6 Updated by Ewoud Kohl van Wijngaarden 7 months ago

  • Fixed in Releases 3.0.1 added

Also available in: Atom PDF