Skip to content

Commit

Permalink
Merge branch 'main' of github.com:Expensify/App into francois-fixTask…
Browse files Browse the repository at this point in the history
…NewPhoneNumber
  • Loading branch information
francoisl committed Jul 12, 2023
2 parents db4026d + 5bcae0b commit db6fd8f
Show file tree
Hide file tree
Showing 11 changed files with 30 additions and 41 deletions.
2 changes: 1 addition & 1 deletion .eslintrc.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ module.exports = {
extends: ['expensify', 'plugin:storybook/recommended', 'plugin:react-hooks/recommended', 'prettier', 'plugin:react-native-a11y/basic'],
plugins: ['react-hooks', 'react-native-a11y'],
parser: 'babel-eslint',
ignorePatterns: ['!.*', 'src/vendor', '.github/actions/**/index.js', 'desktop/dist/*.js', 'dist/*.js', 'node_modules/.bin/**', '.git/**'],
ignorePatterns: ['!.*', 'src/vendor', '.github/actions/**/index.js', 'desktop/dist/*.js', 'dist/*.js', 'node_modules/.bin/**', 'node_modules/.cache/**', '.git/**'],
env: {
jest: true,
},
Expand Down
38 changes: 15 additions & 23 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -182,7 +182,7 @@
"css-loader": "^6.7.2",
"diff-so-fancy": "^1.3.0",
"dotenv": "^16.0.3",
"electron": "22.3.14",
"electron": "^25.2.0",
"electron-builder": "24.5.0",
"eslint": "^7.6.0",
"eslint-config-expensify": "^2.0.38",
Expand Down
2 changes: 1 addition & 1 deletion src/CONST.js
Original file line number Diff line number Diff line change
Expand Up @@ -1141,7 +1141,7 @@ const CONST = {
CARD_SECURITY_CODE: /^[0-9]{3,4}$/,
CARD_EXPIRATION_DATE: /^(0[1-9]|1[0-2])([^0-9])?([0-9]{4}|([0-9]{2}))$/,
PAYPAL_ME_USERNAME: /^[a-zA-Z0-9]{1,20}$/,
ROOM_NAME: /^#[a-z0-9-]{1,80}$/,
ROOM_NAME: /^#[a-z0-9à-ÿ-]{1,80}$/,

// eslint-disable-next-line max-len, no-misleading-character-class
EMOJIS: /[\p{Extended_Pictographic}\u200d\u{1f1e6}-\u{1f1ff}\u{1f3fb}-\u{1f3ff}\u{e0020}-\u{e007f}\u20E3\uFE0F]|[#*0-9]\uFE0F?\u20E3/gu,
Expand Down
1 change: 1 addition & 0 deletions src/components/Pressable/GenericPressable/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@ const WebGenericPressable = forwardRef((props, ref) => (
aria-label={props.accessibilityLabel}
aria-labelledby={props.accessibilityLabelledBy}
aria-valuenow={props.accessibilityValue}
nativeID={props.nativeID || 'no-drag-area'}
/>
));

Expand Down
6 changes: 2 additions & 4 deletions src/components/ReportActionItem/ReportPreview.js
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ import * as IOU from '../../libs/actions/IOU';
import refPropTypes from '../refPropTypes';
import PressableWithoutFeedback from '../Pressable/PressableWithoutFeedback';
import themeColors from '../../styles/themes/default';
import reportPropTypes from '../../pages/reportPropTypes';

const propTypes = {
/** All the data of the action */
Expand All @@ -37,10 +38,7 @@ const propTypes = {

/* Onyx Props */
/** chatReport associated with iouReport */
chatReport: PropTypes.shape({
/** Whether the chat report has an outstanding IOU */
hasOutstandingIOU: PropTypes.bool.isRequired,
}),
chatReport: reportPropTypes,

/** Active IOU Report for current report */
iouReport: PropTypes.shape({
Expand Down
4 changes: 2 additions & 2 deletions src/components/SettlementButton.js
Original file line number Diff line number Diff line change
Expand Up @@ -94,7 +94,7 @@ class SettlementButton extends React.Component {
value: CONST.IOU.PAYMENT_TYPE.PAYPAL_ME,
},
[CONST.IOU.PAYMENT_TYPE.ELSEWHERE]: {
text: isExpenseReport ? this.props.translate('iou.payExpenseElsewhere') : this.props.translate('iou.settleElsewhere'),
text: this.props.translate('iou.payElsewhere'),
icon: Expensicons.Cash,
value: CONST.IOU.PAYMENT_TYPE.ELSEWHERE,
},
Expand Down Expand Up @@ -127,7 +127,7 @@ class SettlementButton extends React.Component {
return [
{
...paymentMethods[paymentMethod],
text: paymentMethod === CONST.IOU.PAYMENT_TYPE.ELSEWHERE ? this.props.translate('iou.payExpenseElsewhere') : this.props.translate('iou.pay'),
text: paymentMethod === CONST.IOU.PAYMENT_TYPE.ELSEWHERE ? this.props.translate('iou.payElsewhere') : this.props.translate('iou.pay'),
},
];
}
Expand Down
3 changes: 1 addition & 2 deletions src/languages/en.js
Original file line number Diff line number Diff line change
Expand Up @@ -353,8 +353,7 @@ export default {
settledElsewhere: 'Paid elsewhere',
settledPaypalMe: 'Paid using Paypal.me',
settleExpensify: 'Pay with Expensify',
settleElsewhere: "I'll settle up elsewhere",
payExpenseElsewhere: 'Pay elsewhere',
payElsewhere: 'Pay elsewhere',
settlePaypalMe: 'Pay with PayPal.me',
requestAmount: ({amount}) => `request ${amount}`,
splitAmount: ({amount}) => `split ${amount}`,
Expand Down
3 changes: 1 addition & 2 deletions src/languages/es.js
Original file line number Diff line number Diff line change
Expand Up @@ -352,8 +352,7 @@ export default {
settledElsewhere: 'Pagado de otra forma',
settledPaypalMe: 'Pagado con PayPal.me',
settleExpensify: 'Pagar con Expensify',
settleElsewhere: 'Voy a pagar de otra forma',
payExpenseElsewhere: 'Pagar de otra forma',
payElsewhere: 'Pagar de otra forma',
settlePaypalMe: 'Pagar con PayPal.me',
requestAmount: ({amount}) => `solicitar ${amount}`,
splitAmount: ({amount}) => `dividir ${amount}`,
Expand Down
6 changes: 1 addition & 5 deletions src/styles/styles.js
Original file line number Diff line number Diff line change
Expand Up @@ -25,10 +25,6 @@ import cursor from './utilities/cursor';
import userSelect from './utilities/userSelect';
import textUnderline from './utilities/textUnderline';

function getTransparentColor(color, transparency = '') {
return `${color}${transparency}`;
}

const picker = {
backgroundColor: themeColors.transparent,
color: themeColors.text,
Expand Down Expand Up @@ -3388,7 +3384,7 @@ const styles = {
linkPreviewWrapper: {
marginTop: 16,
borderLeftWidth: 4,
borderLeftColor: getTransparentColor(themeColors.inverse, 33),
borderLeftColor: themeColors.border,
paddingLeft: 12,
},

Expand Down
4 changes: 4 additions & 0 deletions tests/unit/ValidationUtilsTest.js
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,10 @@ describe('ValidationUtils', () => {
test('room name with lowercase letters, numbers, and dashes', () => {
expect(ValidationUtils.isValidRoomName('#this-is-a-room1')).toBe(true);
});

test('room name with spanish Accented letters and dashes', () => {
expect(ValidationUtils.isValidRoomName('#sala-de-opinión')).toBe(true);
});
});

describe('isValidWebsite', () => {
Expand Down

0 comments on commit db6fd8f

Please sign in to comment.