Project

General

Profile

Feature #30288

replace the existing CPU/Cores/Socket input

Added by yifat makias 12 months ago. Updated 2 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Web Interface
Target version:
-
Difficulty:
Triaged:
No
Bugzilla link:

Description

In order to eventually replace the jquery-ui-spinner, there is a need to replace the existing CPU/Cores/Socket input with the new react component.


Related issues

Related to Foreman - Feature #21918: Remove jquery-ui-spinner and move to a react implementationDuplicate2017-12-10
Related to Foreman - Feature #29900: Add cpu/cores/socket react componentClosed
Blocks Foreman - Refactor #21641: Drop jquery-ui spinners in favor of React imlementationClosed

Associated revisions

Revision 22fa1637 (diff)
Added by yifat makias 3 months ago

Fixes #30288 - replace CPU/Cores/socket input to react comp

Revision 361f3d61 (diff)
Added by Ondřej Ezr 2 months ago

Refs #30288 - improves CPUs counter input

In 22fa163731b38f597e4f96e0c0b345518a360b37 we've introduced React
component to implement the counter funcionality.
This uses the old `counter_f` form helper to render the React component
and effectivelly remove our dependency on the ui.spinner library.

History

#1 Updated by yifat makias 12 months ago

  • Related to Feature #21918: Remove jquery-ui-spinner and move to a react implementation added

#2 Updated by yifat makias 12 months ago

  • Related to Feature #29900: Add cpu/cores/socket react component added

#3 Updated by The Foreman Bot 11 months ago

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

#4 Updated by Ondřej Ezr 3 months ago

  • Blocks Refactor #21641: Drop jquery-ui spinners in favor of React imlementation added

#5 Updated by The Foreman Bot 3 months ago

  • Fixed in Releases 2.5.0 added

#6 Updated by yifat makias 3 months ago

  • Status changed from Ready For Testing to Closed

#7 Updated by The Foreman Bot 2 months ago

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

Also available in: Atom PDF