Refactor #17972
Trends JS is overly complicated, form & helper are outdated and inefficient
Pull request:
Fixed in Releases:
Found in Releases:
Description
JS code is way too complex, the form and its helpers are only used for new trends but still check if this is a new trend and are inefficient in general.
Related issues
Associated revisions
Refs #17972 - remove trends.js from assets initializer
History
#1
Updated by The Foreman Bot over 5 years ago
- Status changed from Assigned to Ready For Testing
- Pull request https://github.com/theforeman/foreman/pull/4162 added
#2
Updated by Anonymous over 5 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset 1d7dcde35468efa61176b69afdd948277ec32204.
#3
Updated by Dominic Cleal over 5 years ago
- Legacy Backlogs Release (now unused) set to 209
#4
Updated by Ohad Levy over 5 years ago
- Related to Bug #18033: unable to create a new trend added
#5
Updated by The Foreman Bot over 5 years ago
- Pull request https://github.com/theforeman/foreman/pull/4184 added
#6
Updated by Daniel Lobato Garcia over 5 years ago
- Target version set to 169
#7
Updated by Brad Buckingham over 5 years ago
- Target version deleted (
169)
#8
Updated by Daniel Lobato Garcia over 5 years ago
- Target version set to 1.11.0
#9
Updated by Ohad Levy about 4 years ago
- Related to Task #23451: move all javascript/css assets to webpack added
Fixes #17972 - Simplify new trend form js and helpers
Migrated the js to webpack as well.