diff options
author | Tristan Cacqueray <tdecacqu@redhat.com> | 2018-12-02 06:15:55 +0000 |
---|---|---|
committer | Tristan Cacqueray <tdecacqu@redhat.com> | 2018-12-06 07:21:37 +0000 |
commit | 839fe0cceb9f1fb7cb965004bc33b8f2527d0c4e (patch) | |
tree | b4f029b7a9d28026ca3d46149835ee3d9fab56ba /web/src/containers/jobs/Jobs.jsx | |
parent | 833dbd257b8c5dbb4721fc74fa794ab63e4f1557 (diff) | |
download | zuul-839fe0cceb9f1fb7cb965004bc33b8f2527d0c4e.tar.gz |
web: refactor jobs page to use a reducer
This change updates the jobs page component to dispatch reducer
action instead of direct axios call. This enables using the generic
error reducers as well as keeping the jobs list in the store to
avoid repeated query. This change also refactor the Job list into
a container and extends the common refresh component.
Change-Id: I7f44e83d6bea8ee915c81d7ba9afd6d9a4c89d38
Diffstat (limited to 'web/src/containers/jobs/Jobs.jsx')
-rw-r--r-- | web/src/containers/jobs/Jobs.jsx | 84 |
1 files changed, 84 insertions, 0 deletions
diff --git a/web/src/containers/jobs/Jobs.jsx b/web/src/containers/jobs/Jobs.jsx new file mode 100644 index 000000000..36a414f71 --- /dev/null +++ b/web/src/containers/jobs/Jobs.jsx @@ -0,0 +1,84 @@ +// Copyright 2018 Red Hat, Inc +// +// Licensed under the Apache License, Version 2.0 (the "License"); you may +// not use this file except in compliance with the License. You may obtain +// a copy of the License at +// +// http://www.apache.org/licenses/LICENSE-2.0 +// +// Unless required by applicable law or agreed to in writing, software +// distributed under the License is distributed on an "AS IS" BASIS, WITHOUT +// WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the +// License for the specific language governing permissions and limitations +// under the License. + +import * as React from 'react' +import PropTypes from 'prop-types' +import { connect } from 'react-redux' +import { Link } from 'react-router-dom' +import { Table } from 'patternfly-react' + + +class JobsList extends React.Component { + static propTypes = { + tenant: PropTypes.object, + jobs: PropTypes.array, + } + + render () { + const { jobs } = this.props + + const headerFormat = value => <Table.Heading>{value}</Table.Heading> + const cellFormat = (value) => ( + <Table.Cell>{value}</Table.Cell>) + const cellJobFormat = (value) => ( + <Table.Cell> + <Link to={this.props.tenant.linkPrefix + '/job/' + value}> + {value} + </Link> + </Table.Cell>) + const cellBuildFormat = (value) => ( + <Table.Cell> + <Link to={this.props.tenant.linkPrefix + '/builds?job_name=' + value}> + builds + </Link> + </Table.Cell>) + const columns = [] + const myColumns = ['name', 'description', 'Last builds'] + myColumns.forEach(column => { + let formatter = cellFormat + let prop = column + if (column === 'name') { + formatter = cellJobFormat + } + if (column === 'Last builds') { + prop = 'name' + formatter = cellBuildFormat + } + columns.push({ + header: {label: column, + formatters: [headerFormat]}, + property: prop, + cell: {formatters: [formatter]} + }) + }) + return ( + <Table.PfProvider + striped + bordered + hover + columns={columns} + > + <Table.Header/> + <Table.Body + rows={jobs} + rowKey="name" + /> + </Table.PfProvider> + ) + } +} + +export default connect(state => ({ + tenant: state.tenant, +}))(JobsList) |