Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🍒 Cherry pick PR #5759 to staging 🍒 #5855

Merged
merged 3 commits into from
Oct 14, 2021
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions android/app/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -150,8 +150,8 @@ android {
minSdkVersion rootProject.ext.minSdkVersion
targetSdkVersion rootProject.ext.targetSdkVersion
multiDexEnabled rootProject.ext.multiDexEnabled
versionCode 1001010720
versionName "1.1.7-20"
versionCode 1001010721
versionName "1.1.7-21"
}
splits {
abi {
Expand Down
2 changes: 1 addition & 1 deletion ios/NewExpensify/Info.plist
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@
</dict>
</array>
<key>CFBundleVersion</key>
<string>1.1.7.20</string>
<string>1.1.7.21</string>
<key>ITSAppUsesNonExemptEncryption</key>
<false/>
<key>LSApplicationQueriesSchemes</key>
Expand Down
2 changes: 1 addition & 1 deletion ios/NewExpensifyTests/Info.plist
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,6 @@
<key>CFBundleSignature</key>
<string>????</string>
<key>CFBundleVersion</key>
<string>1.1.7.20</string>
<string>1.1.7.21</string>
</dict>
</plist>
2 changes: 1 addition & 1 deletion 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
@@ -1,6 +1,6 @@
{
"name": "new.expensify",
"version": "1.1.7-20",
"version": "1.1.7-21",
"author": "Expensify, Inc.",
"homepage": "https://new.expensify.com",
"description": "New Expensify is the next generation of Expensify: a reimagination of payments based atop a foundation of chat.",
Expand Down
10 changes: 5 additions & 5 deletions src/languages/en.js
Original file line number Diff line number Diff line change
Expand Up @@ -205,7 +205,6 @@ export default {
always: 'Always',
daily: 'Daily',
mute: 'Mute',
members: 'Members',
},
loginField: {
addYourPhoneToSettleViaVenmo: 'Add your phone number to settle up via Venmo.',
Expand Down Expand Up @@ -623,7 +622,6 @@ export default {
common: {
card: 'Issue corporate cards',
workspace: 'Workspace',
edit: 'Edit workspace',
settings: 'General settings',
reimburse: 'Reimburse receipts',
bills: 'Pay bills',
Expand All @@ -633,6 +631,7 @@ export default {
bankAccount: 'Connect bank account',
issueAndManageCards: 'Issue and manage cards',
reconcileCards: 'Reconcile cards',
growlMessageOnSave: 'Your workspace settings were successfully saved!',
},
new: {
newWorkspace: 'New workspace',
Expand Down Expand Up @@ -663,7 +662,7 @@ export default {
fastReimbursementsHappyMembers: 'Fast reimbursements = happy members!',
viewAllReceipts: 'View all receipts',
reimburseReceipts: 'Reimburse receipts',
unlockNextDayReimbursements: 'Unlock next day reimbursements',
unlockNextDayReimbursements: 'Unlock next-day reimbursements',
captureNoVBACopyBeforeEmail: 'Ask your workspace members to forward receipts to ',
captureNoVBACopyAfterEmail: ' and download the Expensify App to track cash expenses on the go.',
unlockNoVBACopy: 'Connect a bank account to reimburse your workspace members online.',
Expand All @@ -683,11 +682,12 @@ export default {
invoiceClientsAndCustomers: 'Invoice clients and customers',
invoiceFirstSectionCopy: 'Send beautiful, professional invoices directly to your clients and customers right from within the Expensify app.',
viewAllInvoices: 'View all invoices',
unlockOnlineInvoicesCollection: 'Unlock online invoices collection',
unlockOnlineInvoiceCollection: 'Unlock online invoice collection',
unlockNoVBACopy: 'Connect your bank account to accept online payments for invoices - by ACH or credit card - to be deposited straight into your account.',
moneyBackInAFlash: 'Money back, in a flash!',
unlockVBACopy: 'You\'re all set to accept payments by ACH or credit card!',
viewUnpaidInvoices: 'View unpaid invoices',
sendInvoice: 'Send invoice',
},
travel: {
unlockConciergeBookingTravel: 'Unlock Concierge travel booking',
Expand All @@ -712,7 +712,7 @@ export default {
nameInputLabel: 'Name',
nameInputHelpText: 'This is the name you will see on your workspace.',
nameIsRequiredError: 'You need to define a name for your workspace',
currencyInputLabel: 'Default Currency',
currencyInputLabel: 'Default currency',
currencyInputHelpText: 'All expenses on this workspace will be converted to this currency.',
save: 'Save',
genericFailureMessage: 'An error occurred updating the workspace, please try again.',
Expand Down
6 changes: 3 additions & 3 deletions src/languages/es.js
Original file line number Diff line number Diff line change
Expand Up @@ -205,7 +205,6 @@ export default {
always: 'Siempre',
daily: 'Cada día',
mute: 'Nunca',
members: 'Miembros',
},
loginField: {
addYourPhoneToSettleViaVenmo: 'Agrega tu número de teléfono para pagar usando Venmo.',
Expand Down Expand Up @@ -625,7 +624,6 @@ export default {
common: {
card: 'Emitir tarjetas corporativas',
workspace: 'Espacio de trabajo',
edit: 'Editar espacio de trabajo',
settings: 'Configuración general',
reimburse: 'Reembolsar recibos',
bills: 'Pagar facturas',
Expand All @@ -635,6 +633,7 @@ export default {
bankAccount: 'Conectar cuenta bancaria',
issueAndManageCards: 'Emitir y gestionar tarjetas',
reconcileCards: 'Reconciliar tarjetas',
growlMessageOnSave: '¡La configuración del espacio de trabajo se ha guardado correctamente!',
},
new: {
newWorkspace: 'Nuevo espacio de trabajo',
Expand Down Expand Up @@ -685,11 +684,12 @@ export default {
invoiceClientsAndCustomers: 'Emite facturas a tus clientes',
invoiceFirstSectionCopy: 'Envía facturas detalladas y profesionales directamente a tus clientes desde la app de Expensify.',
viewAllInvoices: 'Ver facturas emitidas',
unlockOnlineInvoicesCollection: 'Desbloquea el cobro de facturas online',
unlockOnlineInvoiceCollection: 'Desbloquea el cobro de facturas online',
unlockNoVBACopy: 'Conecta tu cuenta bancaria para recibir pagos online de facturas - por transferencia o con tarjeta - directamente en tu cuenta.',
moneyBackInAFlash: '¡Tu dinero de vuelta en un momento!',
unlockVBACopy: '¡Todo listo para recibir pagos por transferencia o con tarjeta!',
viewUnpaidInvoices: 'Ver facturas emitidas pendientes',
sendInvoice: 'Enviar factura',
},
travel: {
unlockConciergeBookingTravel: 'Desbloquea la reserva de viajes con Concierge',
Expand Down
6 changes: 5 additions & 1 deletion src/libs/actions/Policy.js
Original file line number Diff line number Diff line change
Expand Up @@ -299,8 +299,9 @@ function uploadAvatar(file) {
*
* @param {String} policyID
* @param {Object} values
* @param {Boolean} [shouldGrowl]
*/
function update(policyID, values) {
function update(policyID, values, shouldGrowl = false) {
API.UpdatePolicy({policyID, value: JSON.stringify(values), lastModified: null})
.then((policyResponse) => {
if (policyResponse.jsonCode !== 200) {
Expand All @@ -313,6 +314,9 @@ function update(policyID, values) {

const updatedValues = {...values, ...{isPolicyUpdating: false}};
Onyx.merge(`${ONYXKEYS.COLLECTION.POLICY}${policyID}`, updatedValues);
if (shouldGrowl) {
Growl.show(translateLocal('workspace.common.growlMessageOnSave'), CONST.GROWL.SUCCESS, 3000);
}
}).catch(() => {
Onyx.merge(`${ONYXKEYS.COLLECTION.POLICY}${policyID}`, {isPolicyUpdating: false});
const errorMessage = translateLocal('workspace.editor.genericFailureMessage');
Expand Down
2 changes: 1 addition & 1 deletion src/pages/ReportDetailsPage.js
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,7 @@ class ReportDetailsPage extends Component {
this.menuItems = isArchivedRoom(this.props.report) ? []
: [
{
translationKey: 'reportDetailsPage.members',
translationKey: 'common.members',
icon: Users,
subtitle: props.report.participants.length,
action: () => { Navigation.navigate(ROUTES.getReportParticipantsRoute(props.report.reportID)); },
Expand Down
2 changes: 1 addition & 1 deletion src/pages/ReportParticipantsPage.js
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ const ReportParticipantsPage = ({
return (
<ScreenWrapper>
<HeaderWithCloseButton
title={translate(isDefaultRoom(report) ? 'reportDetailsPage.members' : 'common.details')}
title={translate(isDefaultRoom(report) ? 'common.members' : 'common.details')}
onCloseButtonPress={Navigation.dismissModal}
onBackButtonPress={Navigation.dismissModal}
shouldShowBackButton={isDefaultRoom(report)}
Expand Down
2 changes: 1 addition & 1 deletion src/pages/workspace/WorkspaceMembersPage.js
Original file line number Diff line number Diff line change
Expand Up @@ -223,7 +223,7 @@ class WorkspaceMembersPage extends React.Component {
return (
<ScreenWrapper style={[styles.defaultModalContainer]}>
<HeaderWithCloseButton
title={this.props.translate('common.members')}
title={this.props.translate('workspace.common.members')}
onCloseButtonPress={() => Navigation.dismissModal()}
onBackButtonPress={() => Navigation.navigate(ROUTES.getWorkspaceInitialRoute(policyID))}
shouldShowBackButton
Expand Down
4 changes: 2 additions & 2 deletions src/pages/workspace/WorkspaceSettingsPage.js
Original file line number Diff line number Diff line change
Expand Up @@ -114,7 +114,7 @@ class WorkspaceSettingsPage extends React.Component {
const policyID = this.props.policy.id;
const currency = this.state.currency;

Policy.update(policyID, {name, avatarURL, outputCurrency: currency});
Policy.update(policyID, {name, avatarURL, outputCurrency: currency}, true);
}).catch(() => {
Policy.updateLocalPolicyValues(this.props.policy.id, {isPolicyUpdating: false});
});
Expand All @@ -134,7 +134,7 @@ class WorkspaceSettingsPage extends React.Component {

return (
<WorkspacePageWithSections
headerText={this.props.translate('workspace.common.edit')}
headerText={this.props.translate('workspace.common.settings')}
route={this.props.route}
>
{hasVBA => (
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ const WorkspaceInvoicesFirstSection = ({translate, policyID}) => (
icon={MoneyEnvelopeBlue}
menuItems={[
{
title: translate('workspace.common.invoices'),
title: translate('workspace.invoices.sendInvoice'),
onPress: () => openOldDotLink('reports?param={"createInvoice":true}'),
icon: Send,
shouldShowRightIcon: true,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ const WorkspaceInvoicesNoVBAView = ({translate, policyID}) => (
<WorkspaceInvoicesFirstSection policyID={policyID} />

<WorkspaceSection
title={translate('workspace.invoices.unlockOnlineInvoicesCollection')}
title={translate('workspace.invoices.unlockOnlineInvoiceCollection')}
icon={JewelBoxGreen}
menuItems={[
{
Expand Down