Merge branch 'develop' into one-click-demo-account

This commit is contained in:
Ahmed Bouhuolia
2024-08-22 10:48:24 +02:00
11 changed files with 119 additions and 22 deletions

View File

@@ -164,6 +164,10 @@ export class Transformer {
return date ? moment(date).format(this.dateFormat) : ''; return date ? moment(date).format(this.dateFormat) : '';
} }
protected formatDateFromNow(date){
return date ? moment(date).fromNow(true) : '';
}
/** /**
* *
* @param number * @param number

View File

@@ -257,25 +257,25 @@ export default {
name: 'item.field.sell_price', name: 'item.field.sell_price',
fieldType: 'number', fieldType: 'number',
}, },
cost_price: { costPrice: {
name: 'item.field.cost_price', name: 'item.field.cost_price',
fieldType: 'number', fieldType: 'number',
}, },
costAccount: { costAccountId: {
name: 'item.field.cost_account', name: 'item.field.cost_account',
fieldType: 'relation', fieldType: 'relation',
relationModel: 'Account', relationModel: 'Account',
relationImportMatch: ['name', 'code'], relationImportMatch: ['name', 'code'],
importHint: 'Matches the account name or code.', importHint: 'Matches the account name or code.',
}, },
sellAccount: { sellAccountId: {
name: 'item.field.sell_account', name: 'item.field.sell_account',
fieldType: 'relation', fieldType: 'relation',
relationModel: 'Account', relationModel: 'Account',
relationImportMatch: ['name', 'code'], relationImportMatch: ['name', 'code'],
importHint: 'Matches the account name or code.', importHint: 'Matches the account name or code.',
}, },
inventoryAccount: { inventoryAccountId: {
name: 'item.field.inventory_account', name: 'item.field.inventory_account',
fieldType: 'relation', fieldType: 'relation',
relationModel: 'Account', relationModel: 'Account',

View File

@@ -8,7 +8,12 @@ export class CashflowAccountTransformer extends Transformer {
* @returns {string[]} * @returns {string[]}
*/ */
public includeAttributes = (): string[] => { public includeAttributes = (): string[] => {
return ['formattedAmount']; return [
'formattedAmount',
'lastFeedsUpdatedAt',
'lastFeedsUpdatedAtFormatted',
'lastFeedsUpdatedFromNow',
];
}; };
/** /**
@@ -29,7 +34,7 @@ export class CashflowAccountTransformer extends Transformer {
/** /**
* Retrieve formatted account amount. * Retrieve formatted account amount.
* @param {IAccount} invoice * @param {IAccount} invoice
* @returns {string} * @returns {string}
*/ */
protected formattedAmount = (account: IAccount): string => { protected formattedAmount = (account: IAccount): string => {
@@ -37,4 +42,22 @@ export class CashflowAccountTransformer extends Transformer {
currencyCode: account.currencyCode, currencyCode: account.currencyCode,
}); });
}; };
/**
* Retrieves the last feeds update at formatted date.
* @param {IAccount} account
* @returns {string}
*/
protected lastFeedsUpdatedAtFormatted(account: IAccount): string {
return this.formatDate(account.lastFeedsUpdatedAt);
}
/**
* Retrieves the last feeds updated from now.
* @param {IAccount} account
* @returns {string}
*/
protected lastFeedsUpdatedFromNow(account: IAccount): string {
return this.formatDateFromNow(account.lastFeedsUpdatedAt);
}
} }

View File

@@ -43,12 +43,22 @@ export class CreateItem {
itemDTO.sellAccountId itemDTO.sellAccountId
); );
} }
// Validate the income account id existance if the item is sellable.
this.validators.validateIncomeAccountExistance(
itemDTO.sellable,
itemDTO.sellAccountId
);
if (itemDTO.costAccountId) { if (itemDTO.costAccountId) {
await this.validators.validateItemCostAccountExistance( await this.validators.validateItemCostAccountExistance(
tenantId, tenantId,
itemDTO.costAccountId itemDTO.costAccountId
); );
} }
// Validate the cost account id existance if the item is purchasable.
this.validators.validateCostAccountExistance(
itemDTO.purchasable,
itemDTO.costAccountId
);
if (itemDTO.inventoryAccountId) { if (itemDTO.inventoryAccountId) {
await this.validators.validateItemInventoryAccountExistance( await this.validators.validateItemInventoryAccountExistance(
tenantId, tenantId,

View File

@@ -55,6 +55,11 @@ export class EditItem {
itemDTO.categoryId itemDTO.categoryId
); );
} }
// Validate the income account id existance if the item is sellable.
this.validators.validateIncomeAccountExistance(
itemDTO.sellable,
itemDTO.sellAccountId
);
// Validate the sell account existance on the storage. // Validate the sell account existance on the storage.
if (itemDTO.sellAccountId) { if (itemDTO.sellAccountId) {
await this.validators.validateItemSellAccountExistance( await this.validators.validateItemSellAccountExistance(
@@ -62,6 +67,11 @@ export class EditItem {
itemDTO.sellAccountId itemDTO.sellAccountId
); );
} }
// Validate the cost account id existance if the item is purchasable.
this.validators.validateCostAccountExistance(
itemDTO.purchasable,
itemDTO.costAccountId
);
// Validate the cost account existance on the storage. // Validate the cost account existance on the storage.
if (itemDTO.costAccountId) { if (itemDTO.costAccountId) {
await this.validators.validateItemCostAccountExistance( await this.validators.validateItemCostAccountExistance(

View File

@@ -85,6 +85,42 @@ export class ItemsValidators {
} }
} }
/**
* Validates income account existance.
* @param {number|null} sellable - Detarmines if the item sellable.
* @param {number|null} incomeAccountId - Income account id.
* @throws {ServiceError(ERRORS.INCOME_ACCOUNT_REQUIRED_WITH_SELLABLE_ITEM)}
*/
public validateIncomeAccountExistance(
sellable?: boolean,
incomeAccountId?: number
) {
if (sellable && !incomeAccountId) {
throw new ServiceError(
ERRORS.INCOME_ACCOUNT_REQUIRED_WITH_SELLABLE_ITEM,
'Income account is require with sellable item.'
);
}
}
/**
* Validates the cost account existance.
* @param {boolean|null} purchasable - Detarmines if the item purchasble.
* @param {number|null} costAccountId - Cost account id.
* @throws {ServiceError(ERRORS.COST_ACCOUNT_REQUIRED_WITH_PURCHASABLE_ITEM)}
*/
public validateCostAccountExistance(
purchasable: boolean,
costAccountId?: number
) {
if (purchasable && !costAccountId) {
throw new ServiceError(
ERRORS.COST_ACCOUNT_REQUIRED_WITH_PURCHASABLE_ITEM,
'The cost account is required with purchasable item.'
);
}
}
/** /**
* Validate item inventory account existance and type. * Validate item inventory account existance and type.
* @param {number} tenantId * @param {number} tenantId

View File

@@ -26,6 +26,11 @@ export const ERRORS = {
PURCHASE_TAX_RATE_NOT_FOUND: 'PURCHASE_TAX_RATE_NOT_FOUND', PURCHASE_TAX_RATE_NOT_FOUND: 'PURCHASE_TAX_RATE_NOT_FOUND',
SELL_TAX_RATE_NOT_FOUND: 'SELL_TAX_RATE_NOT_FOUND', SELL_TAX_RATE_NOT_FOUND: 'SELL_TAX_RATE_NOT_FOUND',
INCOME_ACCOUNT_REQUIRED_WITH_SELLABLE_ITEM:
'INCOME_ACCOUNT_REQUIRED_WITH_SELLABLE_ITEM',
COST_ACCOUNT_REQUIRED_WITH_PURCHASABLE_ITEM:
'COST_ACCOUNT_REQUIRED_WITH_PURCHASABLE_ITEM',
}; };
export const DEFAULT_VIEW_COLUMNS = []; export const DEFAULT_VIEW_COLUMNS = [];

View File

@@ -62,6 +62,7 @@ export function BankAccount({
balance, balance,
loading = false, loading = false,
updatedBeforeText, updatedBeforeText,
uncategorizedTransactionsCount,
...restProps ...restProps
}) { }) {
return ( return (
@@ -77,17 +78,19 @@ export function BankAccount({
</BankAccountHeader> </BankAccountHeader>
<BankAccountMeta> <BankAccountMeta>
{false && ( {uncategorizedTransactionsCount > 0 && (
<BankAccountMetaLine <BankAccountMetaLine
title={intl.get('cash_flow.transactions_for_review')} title={intl.get('cash_flow.transactions_for_review')}
value={'0'} value={uncategorizedTransactionsCount}
className={clsx({ [Classes.SKELETON]: loading })}
/>
)}
{updatedBeforeText && (
<BankAccountMetaLine
title={updatedBeforeText}
className={clsx({ [Classes.SKELETON]: loading })} className={clsx({ [Classes.SKELETON]: loading })}
/> />
)} )}
<BankAccountMetaLine
title={updatedBeforeText}
className={clsx({ [Classes.SKELETON]: loading })}
/>
</BankAccountMeta> </BankAccountMeta>
<BankAccountBalance amount={balance} loading={loading} /> <BankAccountBalance amount={balance} loading={loading} />

View File

@@ -44,9 +44,9 @@ export const SubscriptionPlans = [
}, },
{ text: 'Unlimited User Seats' }, { text: 'Unlimited User Seats' },
], ],
monthlyPrice: '$10', monthlyPrice: '$20',
monthlyPriceLabel: 'Per month', monthlyPriceLabel: 'Per month',
annuallyPrice: '$7.5', annuallyPrice: '$15',
annuallyPriceLabel: 'Per month', annuallyPriceLabel: 'Per month',
monthlyVariantId: '446152', monthlyVariantId: '446152',
// monthlyVariantId: '450016', // monthlyVariantId: '450016',
@@ -78,9 +78,9 @@ export const SubscriptionPlans = [
{ text: 'Smart Financial Reports' }, { text: 'Smart Financial Reports' },
{ text: 'Advanced Inventory Reports' }, { text: 'Advanced Inventory Reports' },
], ],
monthlyPrice: '$20', monthlyPrice: '$40',
monthlyPriceLabel: 'Per month', monthlyPriceLabel: 'Per month',
annuallyPrice: '$15', annuallyPrice: '$30',
annuallyPriceLabel: 'Per month', annuallyPriceLabel: 'Per month',
// monthlyVariantId: '450028', // monthlyVariantId: '450028',
monthlyVariantId: '446155', monthlyVariantId: '446155',
@@ -101,9 +101,9 @@ export const SubscriptionPlans = [
}, },
{ text: 'Analysis Cost Center' }, { text: 'Analysis Cost Center' },
], ],
monthlyPrice: '$25', monthlyPrice: '$55',
monthlyPriceLabel: 'Per month', monthlyPriceLabel: 'Per month',
annuallyPrice: '$19', annuallyPrice: '$40',
annuallyPriceLabel: 'Per month', annuallyPriceLabel: 'Per month',
featured: true, featured: true,
// monthlyVariantId: '450031', // monthlyVariantId: '450031',
@@ -128,9 +128,9 @@ export const SubscriptionPlans = [
hint: 'Track the organization inventory in multiple warehouses and transfer goods between them.', hint: 'Track the organization inventory in multiple warehouses and transfer goods between them.',
}, },
], ],
monthlyPrice: '$40', monthlyPrice: '$60',
monthlyPriceLabel: 'Per month', monthlyPriceLabel: 'Per month',
annuallyPrice: '$30', annuallyPrice: '$45',
annuallyPriceLabel: 'Per month', annuallyPriceLabel: 'Per month',
// monthlyVariantId: '450024', // monthlyVariantId: '450024',
monthlyVariantId: '446167', monthlyVariantId: '446167',

View File

@@ -43,6 +43,7 @@ export function useExcludedTransactionsColumns() {
{ {
Header: 'Description', Header: 'Description',
accessor: descriptionAccessor, accessor: descriptionAccessor,
textOverview: true,
}, },
{ {
Header: 'Payee', Header: 'Payee',

View File

@@ -123,7 +123,12 @@ function CashflowBankAccount({
code={account.code} code={account.code}
balance={!isNull(account.amount) ? account.formatted_amount : '-'} balance={!isNull(account.amount) ? account.formatted_amount : '-'}
type={account.account_type} type={account.account_type}
updatedBeforeText={getUpdatedBeforeText(account.createdAt)} updatedBeforeText={
account.last_feeds_updated_from_now
? `Updated ${account.last_feeds_updated_from_now} ago`
: ''
}
uncategorizedTransactionsCount={account.uncategorized_transactions}
/> />
</CashflowAccountAnchor> </CashflowAccountAnchor>
</ContextMenu2> </ContextMenu2>