#67 Added spin loader to wizard section i.e /wizard route
Closed 2 years ago by lbrabec. Opened 2 years ago by kunalprakash1309.
fedora-qa/ kunalprakash1309/landingpage wizard-spinner  into  master

file modified
+2
@@ -15,6 +15,7 @@ 

      actions: [],

      providers: [],

      all_actions: [],

+     isLoading: true,

    },

  }

  
@@ -33,6 +34,7 @@ 

            ...state.wizard,

            providers: action.payload.providers,

            all_actions: action.payload.actions,

+           isLoading: false,

          },

        }

  

file modified
+1 -9
@@ -17,14 +17,6 @@ 

    }

  

    componentDidMount() {

-     fetch(oraculumApiUrl_v1 + "actions/all")

-       .then((resp) => resp.json())

-       .then((data) => {

-         this.setState({

-           providers: data.providers,

-           all_actions: data.actions,

-         })

-       })

  

      this.props.dispatch(loadWizardData())

    }
@@ -35,7 +27,7 @@ 

  

    render() {

      return (

-       <Layout>

+       <Layout loading={this.props.isLoading}>

          <div className="bluebox">

            <Container>

              <WizardForm

Whenever we load /landingpage there is spin loader on right side of the navbar. But when we load /wizard there is no such spin loader.

This PR solves this issue and makes the UI more better.

Metadata Update from @lbrabec:
- Pull-request tagged with: review done

2 years ago

Thanks for PR, however this functionality was part of #32, which is now merged. Closing this one.

Pull-Request has been closed by lbrabec

2 years ago