Skip to content

Commit

Permalink
Merge pull request #30831 from DylanDylann/fix/30304-adress-zipcode-i…
Browse files Browse the repository at this point in the history
…s-saved-when-chaging-country

Fix/30304: zipcode does not clear when country change
  • Loading branch information
lakchote authored Nov 6, 2023
2 parents b5d3f90 + 040c8a7 commit 8d17c95
Showing 1 changed file with 14 additions and 3 deletions.
17 changes: 14 additions & 3 deletions src/pages/settings/Profile/PersonalDetails/AddressPage.js
Original file line number Diff line number Diff line change
Expand Up @@ -81,6 +81,7 @@ function AddressPage({privatePersonalDetails, route}) {
const [street1, street2] = (address.street || '').split('\n');
const [state, setState] = useState(address.state);
const [city, setCity] = useState(address.city);
const [zipcode, setZipcode] = useState(address.zip);

useEffect(() => {
if (!address) {
Expand All @@ -89,6 +90,7 @@ function AddressPage({privatePersonalDetails, route}) {
setState(address.state);
setCurrentCountry(address.country);
setCity(address.city);
setZipcode(address.zip);
}, [address]);

/**
Expand Down Expand Up @@ -137,20 +139,28 @@ function AddressPage({privatePersonalDetails, route}) {
}, []);

const handleAddressChange = useCallback((value, key) => {
if (key !== 'country' && key !== 'state' && key !== 'city') {
if (key !== 'country' && key !== 'state' && key !== 'city' && key !== 'zipPostCode') {
return;
}
if (key === 'country') {
setCurrentCountry(value);
setState('');
setCity('');
setZipcode('');
return;
}
if (key === 'state') {
setState(value);
setCity('');
setZipcode('');
return;
}
if (key === 'city') {
setCity(value);
setZipcode('');
return;
}
setCity(value);
setZipcode(value);
}, []);

useEffect(() => {
Expand Down Expand Up @@ -254,9 +264,10 @@ function AddressPage({privatePersonalDetails, route}) {
accessibilityLabel={translate('common.zipPostCode')}
accessibilityRole={CONST.ACCESSIBILITY_ROLE.TEXT}
autoCapitalize="characters"
defaultValue={address.zip || ''}
value={zipcode || ''}
maxLength={CONST.BANK_ACCOUNT.MAX_LENGTH.ZIP_CODE}
hint={zipFormat}
onValueChange={handleAddressChange}
/>
</Form>
)}
Expand Down

0 comments on commit 8d17c95

Please sign in to comment.