Skip to content

Commit

Permalink
Merge pull request #523 from fedspendingtransparency/hottest-fix-enab…
Browse files Browse the repository at this point in the history
…le-dot-agency-page

Hottest Fix - DOT Agency Page
  • Loading branch information
kevinli-work authored Nov 17, 2017
2 parents 1bb4dc3 + eabd300 commit 11bcb4a
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 15 deletions.
2 changes: 0 additions & 2 deletions src/js/components/agencyLanding/table/TableRow.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,6 @@ export default class TableRow extends React.PureComponent {
rowClass = 'row-odd';
}
const cells = this.props.columns.map((column) => {
// BODGE: Disable Department of Transportation in agency list
if (column.columnName === 'agency_name') {
// show the agency link cell
return (
Expand All @@ -34,7 +33,6 @@ export default class TableRow extends React.PureComponent {
name={this.props.agency.agency_name}
id={this.props.agency.agency_id}
agencySearchString={this.props.agencySearchString}
disabled={this.props.agency.agency_name === 'Department of Transportation (DOT)'}
column={column.columnName} />
</td>
);
Expand Down
12 changes: 1 addition & 11 deletions src/js/components/agencyLanding/table/cells/AgencyLinkCell.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,7 @@ const propTypes = {
rowIndex: PropTypes.number,
column: PropTypes.string,
id: PropTypes.number,
agencySearchString: PropTypes.string,
disabled: PropTypes.bool
agencySearchString: PropTypes.string
};

export default class AgencyLinkCell extends React.Component {
Expand All @@ -30,15 +29,6 @@ export default class AgencyLinkCell extends React.Component {
));
}

// BODGE: Disable specific agencies in agency list
if (this.props.disabled) {
return (<div className={`agency-link-cell column-${this.props.column}`}>
<div className="cell-content">
{name}
</div>
</div>);
}

return (
<div className={`agency-link-cell column-${this.props.column}`}>
<div className="cell-content">
Expand Down
3 changes: 1 addition & 2 deletions src/js/components/explorer/detail/header/DetailHeader.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -58,8 +58,7 @@ const heading = (type, title, id) => {
</h2>
);
}
else if (type === 'Agency' && title !== 'Department of Transportation') {
// BODGE: Disable link to Department of Transportation
else if (type === 'Agency') {
return (
<h2>
<a href={`/#/agency/${id}`}>{title}</a>
Expand Down

0 comments on commit 11bcb4a

Please sign in to comment.