Skip to content
This repository has been archived by the owner on Feb 1, 2024. It is now read-only.

Commit

Permalink
Merge pull request #1722 from open-apparel-registry/eh/remove-native-…
Browse files Browse the repository at this point in the history
…language-field
  • Loading branch information
jwalgran committed Mar 18, 2022
2 parents 688090c + a5edce1 commit 7e06f60
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 24 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
- Update email copy for list completion [#1709](https://github.com/open-apparel-registry/open-apparel-registry/pull/1709)
- Remove the redundant workers suffix when showing value [#1715](https://github.com/open-apparel-registry/open-apparel-registry/pull/1715/files)
- Update copy on list pages [#1719](https://github.com/open-apparel-registry/open-apparel-registry/pull/1719)
- Remove native language name from filter sidebar [#1722](https://github.com/open-apparel-registry/open-apparel-registry/pull/1722)

### Deprecated

Expand Down
24 changes: 0 additions & 24 deletions src/app/src/components/FilterSidebarSearchTab.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -114,7 +114,6 @@ const FACILITY_TYPE = 'FACILITY_TYPE';
const PROCESSING_TYPE = 'PROCESSING_TYPE';
const PRODUCT_TYPE = 'PRODUCT_TYPE';
const NUMBER_OF_WORKERS = 'NUMBER_OF_WORKERS';
const NATIVE_LANGUAGE_NAME = 'NATIVE_LANGUAGE_NAME';

const mapFacilityTypeOptions = (fPTypes, pTypes) => {
let fTypes = [];
Expand Down Expand Up @@ -181,8 +180,6 @@ function FilterSidebarSearchTab({
updateProductType,
numberOfWorkers,
updateNumberOfWorkers,
nativeLanguageName,
updateNativeLanguageName,
combineContributors,
updateCombineContributors,
fetchingFacilities,
Expand Down Expand Up @@ -210,7 +207,6 @@ function FilterSidebarSearchTab({
processingType,
productType,
numberOfWorkers,
nativeLanguageName,
];

const allFields = extendedFields.concat([
Expand Down Expand Up @@ -767,25 +763,6 @@ function FilterSidebarSearchTab({
disabled={fetchingOptions || fetchingFacilities}
/>
</div>
<div
className="form__field"
style={{ marginBottom: '10px' }}
>
<InputLabel
htmlFor={NATIVE_LANGUAGE_NAME}
className="form__label"
>
Native Language Name
</InputLabel>
<TextField
id={NATIVE_LANGUAGE_NAME}
placeholder="Native Language Facility Name"
className="full-width margin-bottom-16 form__text-input"
value={nativeLanguageName}
onChange={updateNativeLanguageName}
onKeyPress={submitFormOnEnterKeyPress}
/>
</div>
</ShowOnly>
<div className="form__action">{searchResetButtonGroup()}</div>
<div className="form__field">
Expand Down Expand Up @@ -849,7 +826,6 @@ FilterSidebarSearchTab.propTypes = {
processingType: processingTypeOptionsPropType.isRequired,
productType: productTypeOptionsPropType.isRequired,
numberOfWorkers: numberOfWorkerOptionsPropType.isRequired,
nativeLanguageName: string.isRequired,
combineContributors: string.isRequired,
ppe: string.isRequired,
fetchingFacilities: bool.isRequired,
Expand Down

0 comments on commit 7e06f60

Please sign in to comment.