Project

General

Profile

Refactor #20652

move ovirt js code into webpack

Added by Ohad Levy almost 2 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Target version:
Difficulty:
Triaged:
Bugzilla link:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

This simply moves all ovirt related JS code under webpack control.


Related issues

Related to Foreman - Bug #22420: compute resource vms and images are not shown if you navigate from another pageClosed2018-01-25
Related to Docker - Bug #22532: Test connection on docker compute resource shows ReferenceError: testConnection is not definedNew
Related to Foreman - Task #23451: move all javascript/css assets to webpackNew2018-04-30

Associated revisions

Revision 9234d6c5 (diff)
Added by Ohad Levy almost 2 years ago

fixes #20652 - moved compute_resource js code to webpack

this creates a new namespace under tfm, tfm.computeResource
where relevant functions are expoed under the compute resource
namespace, for example:

tfm.computeResource.ovirt.termplateSelected function

Additional common functions are also exposed under the same namespace:

tfm.computeResource.providerSelected

I left the host form NIC code out of this PR as it actually requires
code changes, changes in this PR are comsetic (mostly linting) and
convertion to ES6.

History

#1 Updated by The Foreman Bot almost 2 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Ohad Levy
  • Pull request https://github.com/theforeman/foreman/pull/4757 added

#2 Updated by Tomer Brisker almost 2 years ago

  • Legacy Backlogs Release (now unused) set to 296

#3 Updated by Ohad Levy almost 2 years ago

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

#4 Updated by The Foreman Bot over 1 year ago

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

#5 Updated by Tomer Brisker over 1 year ago

  • Pull request deleted (https://github.com/theforeman/foreman/pull/5074)

#6 Updated by Ohad Levy over 1 year ago

  • Related to Bug #22420: compute resource vms and images are not shown if you navigate from another page added

#7 Updated by Ohad Levy over 1 year ago

  • Related to Bug #22532: Test connection on docker compute resource shows ReferenceError: testConnection is not defined added

#8 Updated by Ohad Levy over 1 year ago

  • Related to Task #23451: move all javascript/css assets to webpack added

Also available in: Atom PDF