Project

General

Profile

Bug #23486

There are js warnings when a modal (react) opens

Added by Amir Fefer almost 5 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Amir Fefer
Category:
JavaScript stack
Target version:
1.18.1
Difficulty:
Triaged:
No
Bugzilla link:
1599875
Pull request:
https://github.com/theforeman/foreman/pull/5929, https://github.com/theforeman/foreman/pull/5524
Fixed in Releases:
1.18.1, 1.19.0
Found in Releases:
Red Hat JIRA:

Related issues

Related to Katello - Bug #24354: Black background instead of transparency on subscription related modalsClosed
Related to Statistics - Bug #24540: Statistics headers are missingResolved

Associated revisions

Revision 9fed8848 (diff)
Added by Amir Fefer almost 5 years ago

Fixes #23486 - remove js warnings when a modal opens

History

#1 Updated by Amir Fefer almost 5 years ago

  • Subject changed from Remove js warnings when a modal (react) opens to There are js warnings when a modal (react) opens

#2 Updated by The Foreman Bot almost 5 years ago

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

#3 Updated by Amir Fefer almost 5 years ago

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

#4 Updated by Marek Hulán almost 5 years ago

  • Legacy Backlogs Release (now unused) set to 353

#5 Updated by Walden Raines over 4 years ago

  • Triaged set to No
  • Target version changed from 1.19.0 to 1.18.1

We need this in 1.18 as without it we get this a black background on modals in katello:

#6 Updated by Walden Raines over 4 years ago

  • Related to Bug #24354: Black background instead of transparency on subscription related modals added

#7 Updated by Walden Raines over 4 years ago

Also, 1.18 is already using 0.32.1 since that's what is in the repo: https://yum.theforeman.org/releases/1.18/el7/x86_64/

#8 Updated by Zach Huntington-Meath over 4 years ago

  • Bugzilla link set to 1599875

#9 Updated by Ohad Levy over 4 years ago

  • Related to Bug #24540: Statistics headers are missing added

#10 Updated by The Foreman Bot over 4 years ago

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

#11 Updated by Tomer Brisker over 4 years ago

  • Fixed in Releases 1.18.1 added

Also available in: Atom PDF