Source variables provided at build time

awx/ui_next/public/static/media/default.strings.json

@@ -0,0 +1,4 @@
+{
+    "BRAND_NAME": "AWX",
+    "PENDO_API_KEY": ""
+}

awx/ui_next/src/api/models/Root.js

@@ -25,6 +25,14 @@ class Root extends Base {
   logout() {
     return this.http.get(`${this.baseUrl}logout/`);
   }
+
+  readAssetVariables() {
+    // TODO: There's better ways of doing this. Build tools, scripts,
+    // automation etc. should relocate this variable file to an importable
+    // location in src prior to building. That said, a raw http call
+    // works for now.
+    return this.http.get('/static/media/default.strings.json');
+  }
 }
 
 export default Root;

awx/ui_next/src/screens/Login/Login.jsx

@@ -5,7 +5,6 @@ import { t } from '@lingui/macro';
 import styled from 'styled-components';
 import { LoginForm, LoginPage as PFLoginPage } from '@patternfly/react-core';
 import { RootAPI } from '../../api';
-import { BrandName } from '../../variables';
 
 const loginLogoSrc = '/static/media/logo-login.svg';
 
@@ -28,6 +27,7 @@ class AWXLogin extends Component {
       isLoading: true,
       logo: null,
       loginInfo: null,
+      brandName: null,
     };
 
     this.handleChangeUsername = this.handleChangeUsername.bind(this);
@@ -43,16 +43,24 @@ class AWXLogin extends Component {
   async loadCustomLoginInfo() {
     this.setState({ isLoading: true });
     try {
-      const {
+      const [
-        data: { custom_logo, custom_login_info },
+        {
-      } = await RootAPI.read();
+          data: { custom_logo, custom_login_info },
+        },
+        {
+          data: { BRAND_NAME },
+        },
+      ] = await Promise.all([RootAPI.read(), RootAPI.readAssetVariables()]);
       const logo = custom_logo
         ? `data:image/jpeg;${custom_logo}`
         : loginLogoSrc;
-
+      this.setState({
-      this.setState({ logo, loginInfo: custom_login_info });
+        brandName: BRAND_NAME,
+        logo,
+        loginInfo: custom_login_info,
+      });
     } catch (err) {
-      this.setState({ logo: loginLogoSrc });
+      this.setState({ brandName: 'AWX', logo: loginLogoSrc });
     } finally {
       this.setState({ isLoading: false });
     }
@@ -93,6 +101,7 @@ class AWXLogin extends Component {
 
   render() {
     const {
+      brandName,
       hasAuthError,
       hasValidationError,
       username,
@@ -102,10 +111,6 @@ class AWXLogin extends Component {
       loginInfo,
     } = this.state;
     const { alt, i18n, isAuthenticated } = this.props;
-    // Setting BrandName to a variable here is necessary to get the jest tests
-    // passing.  Attempting to use BrandName in the template literal results
-    // in failing tests.
-    const brandName = BrandName;
 
     if (isLoading) {
       return null;
@@ -126,7 +131,11 @@ class AWXLogin extends Component {
       <LoginPage
         brandImgSrc={logo}
         brandImgAlt={alt || brandName}
-        loginTitle={i18n._(t`Welcome to Ansible ${brandName}! Please Sign In.`)}
+        loginTitle={
+          brandName
+            ? i18n._(t`Welcome to Ansible ${brandName}! Please Sign In.`)
+            : ''
+        }
         textContent={loginInfo}
       >
         <LoginForm

awx/ui_next/src/screens/Login/Login.test.jsx

@@ -55,6 +55,11 @@ describe('<Login />', () => {
         custom_logo: 'images/foo.jpg',
       },
     });
+    RootAPI.readAssetVariables.mockResolvedValue({
+      data: {
+        BRAND_NAME: 'AWX',
+      },
+    });
   });
 
   afterEach(() => {