@janiscommerce/app-tracking-shift 1.7.0-beta.2 → 1.7.0-beta.3

package/lib/Formatter.js

@@ -1,4 +1,4 @@
-import {isArray, isEmptyArray, isNumber} from './helpers';
+import {isArray, isEmptyArray, isNumber} from '@janiscommerce/apps-helpers';
 import {INTERNAL_WORKLOGS} from './constant';
 
 class Formatter {

package/lib/OfflineData.js

@@ -1,6 +1,6 @@
+import {isArray, isEmptyArray} from '@janiscommerce/apps-helpers';
 import {OFFLINE_DATA} from './constant';
 import Storage from './db/StorageService';
-import {isArray, isEmptyArray} from './helpers';
 
 class OfflineData {
 	get hasData() {

package/lib/Shift.js

@@ -1,18 +1,10 @@
+import {isArray, isEmptyArray, isNumber} from '@janiscommerce/apps-helpers';
 import StaffService from './StaffApiServices';
 import Storage from './db/StorageService';
 import Crashlytics from './helpers/crashlytics';
 import ShiftWorklogs from './ShiftWorklogs';
 import errorParser from './helpers/errorParser';
-import {
-	isArray,
-	isEmptyArray,
-	isNumber,
-	isValidObject,
-	isApiError,
-	isInternetReachable,
-	isValidDate,
-	parseToISOString,
-} from './helpers';
+import {isApiError, isInternetReachable, isValidDate, parseToISOString} from './helpers';
 import {getStaffAuthorizationData} from './helpers/storage';
 import {
 	SHIFT_ID,
@@ -263,7 +255,7 @@ class Shift {
 		try {
 			Crashlytics.log('openWorkLog:', workLog);
 
-			if (!isValidObject(workLog)) return null;
+			if (!ShiftWorklogs.isValidWorkLog(workLog)) return null;
 
 			if (!this.hasStaffAuthorization) {
 				this._throwAuthorizationError();
@@ -334,7 +326,7 @@ class Shift {
 		try {
 			Crashlytics.log('finishWorkLog:', workLog);
 
-			if (!isValidObject(workLog)) return null;
+			if (!ShiftWorklogs.isValidWorkLog(workLog)) return null;
 
 			if (!this.hasStaffAuthorization) {
 				this._throwAuthorizationError();
@@ -342,7 +334,7 @@ class Shift {
 
 			const currentWorkLog = this.getCurrentWorkLog();
 
-			if (!isValidObject(currentWorkLog)) {
+			if (!ShiftWorklogs.isValidWorkLog(currentWorkLog)) {
 				throw new Error('There is no active worklog to close');
 			}
 
@@ -408,7 +400,7 @@ class Shift {
 
 		return {
 			...(!!id && {id}),
-			...(isValidObject(data) && {...data}),
+			...(ShiftWorklogs.isValidWorkLog(data) && {...data}),
 		};
 	}
 

package/lib/ShiftWorklogs.js

@@ -1,6 +1,12 @@
+import {
+	isObject,
+	isEmptyObject,
+	isArray,
+	isEmptyArray,
+	generateRandomId,
+} from '@janiscommerce/apps-helpers';
 import {INTERNAL_WORKLOGS} from './constant';
 import StaffApiServices from './StaffApiServices';
-import {generateRandomId, isArray, isEmptyArray} from './helpers';
 
 class ShiftWorklogs {
 	async open(params) {
@@ -101,6 +107,10 @@ class ShiftWorklogs {
 			})
 			.filter(Boolean);
 	}
+
+	isValidWorkLog(workLog = {}) {
+		return isObject(workLog) && !isEmptyObject(workLog);
+	}
 }
 
 export default new ShiftWorklogs();

package/lib/StaffApiServices.js

@@ -1,5 +1,5 @@
+import {isString} from '@janiscommerce/apps-helpers';
 import Request from './helpers/request';
-import {isValidString} from './helpers';
 
 class StaffApiServices {
 	constructor() {
@@ -19,7 +19,7 @@ class StaffApiServices {
 				service: this.service,
 				namespace: 'shift-open',
 				body: {
-					...(isValidString(warehouseId) && {warehouseId}),
+					...(isString(warehouseId) && !!warehouseId && {warehouseId}),
 				},
 			});
 		} catch (error) {
@@ -40,7 +40,7 @@ class StaffApiServices {
 				service: this.service,
 				namespace: 'shift-update',
 				body: {
-					...(isValidString(warehouseId) && {warehouseId}),
+					...(isString(warehouseId) && !!warehouseId && {warehouseId}),
 				},
 			});
 		} catch (error) {

package/lib/helpers/index.js

@@ -1,49 +1,30 @@
+import {isNumber, isString} from '@janiscommerce/apps-helpers';
+
 export {default as isApiError} from './isApiError';
 
 export {default as isInternetReachable} from './isInternetReachable';
 
-export const generateRandomId = () => Math.random().toString(32).slice(2);
-
-export const isFunction = (fn) => !!({}.toString.call(fn) === '[object Function]');
-
-export const isObject = (obj) => !!(obj && obj.constructor === Object);
-
-export const isEmptyObject = (obj) => isObject(obj) && !Object.keys(obj).length;
-
-export const promiseWrapper = (promise) =>
-	promise.then((data) => [data, null]).catch((error) => Promise.resolve([null, error]));
-
-export const isArray = (arr) => Array.isArray(arr);
-
-export const isEmptyArray = (arr) => isArray(arr) && !arr.length;
-
 export const reverseArray = (arr) => arr.slice().reverse();
 
-export const isNumber = (num) => typeof num === 'number' && !Number.isNaN(Number(num));
-
-export const isValidObject = (obj) => isObject(obj) && !!Object.keys(obj).length;
-
-export const isValidString = (str) => typeof str === 'string' && str.length > 0;
-
 const ISO_DATE_REGEX = /^\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}\.\d{3}Z$/;
-export const isIsoStringDate = (str) => ISO_DATE_REGEX.test(str);
+const isIsoStringDate = (str) => ISO_DATE_REGEX.test(str);
 
 export const isValidDate = (date = '') => {
 	if (isNumber(date) && date > 0) {
 		const dateTime = new Date(date).getTime();
-		return !Number.isNaN(dateTime);
+		return isNumber(dateTime);
 	}
-	if (!isValidString(date)) return false;
+	if (!isString(date) || !date?.length) return false;
 
 	if (!isIsoStringDate(date)) return false;
 
 	const dateTime = new Date(date).getTime();
-	return !Number.isNaN(dateTime);
+	return isNumber(dateTime);
 };
 
 export const parseToISOString = (date = '') => {
 	if (isNumber(date) && date > 0) return new Date(date).toISOString();
-	if (!isValidString(date)) return '';
+	if (!isString(date) || !date?.length) return '';
 
 	return date;
 };

package/lib/helpers/provider/downloadWorkLogTypes/index.js

@@ -1,9 +1,9 @@
+import {isFunction} from '@janiscommerce/apps-helpers';
 import Shift from '../../../Shift';
 import {getWorkLogTypesData} from '../../storage';
 import {WORKLOG_TYPES_DATA, WORKLOG_TYPES_EXPIRATION_TIME} from '../../../constant';
 import Storage from '../../../db/StorageService';
 import Crashlytics from '../../crashlytics';
-import {isFunction} from '../..';
 import errorParser from '../../errorParser';
 
 const downloadWorkLogTypes = async (onDownloadError) => {

package/lib/helpers/provider/openShift/index.js

@@ -1,7 +1,7 @@
+import {isFunction} from '@janiscommerce/apps-helpers';
 import Crashlytics from '../../crashlytics';
 import getUserId from '../../userInfo/getUserId';
 import Shift from '../../../Shift';
-import {isFunction} from '../..';
 import errorParser from '../../errorParser';
 
 const openShift = async ({onOpenShiftError, warehouseId}) => {

package/lib/provider/ShiftTrackingProvider.js

@@ -1,4 +1,5 @@
 import React, {useEffect, useMemo, useState} from 'react';
+import {promiseWrapper} from '@janiscommerce/apps-helpers';
 import {useStorageValue} from '../hooks/useStorageValue';
 import ShiftTrackingContext from '../context/ShiftTrackingContext';
 import {
@@ -17,8 +18,8 @@ import {
 	STAFF_AUTH,
 	WORKLOG_TYPES_DATA,
 } from '../constant';
-import {isValidObject, promiseWrapper} from '../helpers';
 import Shift from '../Shift';
+import ShiftWorklogs from '../ShiftWorklogs';
 
 /**
  * ShiftTrackingProvider component
@@ -145,11 +146,11 @@ const ShiftTrackingProvider = ({children, additionalInfo = {}, onError = null})
 		const [currentWorkLog = {}] = openWorkLogs;
 		const isExcludedWork = EXCLUDED_WORKLOG_TYPES.includes(currentWorkLog?.referenceId);
 
-		if (isValidObject(currentWorkLog)) {
+		if (ShiftWorklogs.isValidWorkLog(currentWorkLog)) {
 			Shift.setCurrentWorkLog(currentWorkLog);
 		}
 
-		if (isValidObject(currentWorkLog) && !isExcludedWork) {
+		if (ShiftWorklogs.isValidWorkLog(currentWorkLog) && !isExcludedWork) {
 			Shift.status = 'paused';
 		}
 

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@janiscommerce/app-tracking-shift",
-	"version": "1.7.0-beta.2",
+	"version": "1.7.0-beta.3",
 	"main": "lib/index.js",
 	"module": "lib/index.js",
 	"exports": {
@@ -60,6 +60,7 @@
 		"@janiscommerce/app-crashlytics": "^2.1.0",
 		"@janiscommerce/app-request": "^2.6.0",
 		"@janiscommerce/app-storage": "^1.1.0",
+		"@janiscommerce/apps-helpers": "^2.2.0",
 		"@janiscommerce/oauth-native": "^1.10.2",
 		"@testing-library/react": "^16.3.0",
 		"@testing-library/react-native": "^12.0.1",
@@ -81,8 +82,8 @@
 		"react": "18.2.0",
 		"react-native": "0.71.6",
 		"react-native-device-info": "^10.12.0",
-		"react-native-mmkv": "2.12.2",
 		"react-native-inappbrowser-reborn": "^3.7.0",
+		"react-native-mmkv": "2.12.2",
 		"react-test-renderer": "18.2.0"
 	}
 }