admin/bodyshop
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

IO-3020 IO-3036 Remove additional TODOs.

Browse Source
This commit is contained in:
Patrick Fic
2024-11-25 12:29:09 -08:00
parent eacadc01bd
commit 4433f0f57f
12 changed files with 43 additions and 49 deletions
Download Patch File Download Diff File Expand all files Collapse all files

59
client/src/components/jobs-available-table/jobs-available-table.container.jsx
View File

@@ -115,7 +115,7 @@ export function JobsAvailableContainer({ bodyshop, currentUser, insertAuditTrail
// } else {
//IO-539 Check for Parts Rate on PAL for SGI use case.
//TODO:AIO Check that the async function is actually waiting before moving on.
await InstanceRenderManager({
executeFunction: true,
imex: CheckTaxRates,
@@ -568,7 +568,6 @@ async function CheckTaxRates(estData, bodyshop) {
}
function ResolveCCCLineIssues(estData, bodyshop) {
//Find all misc amounts, populate them to the act price.
//TODO Ensure that this doesnt get violated
//This needs to be done before cleansing unq_seq since some misc prices could move over.
estData.joblines.data.forEach((line) => {
if (line.misc_amt && line.misc_amt !== 0) {
@@ -591,34 +590,34 @@ function ResolveCCCLineIssues(estData, bodyshop) {
//Group by line no
// For everything but the first one, strip out the price number in
//TODO: How do we merge these 2 functions together and have it function properly for Rome?
InstanceRenderManager({
executeFunction: true,
args: [],
promanager: () => {
const groupedByLineRef = _.groupBy(estData.joblines.data, "line_ref");
Object.keys(groupedByLineRef).forEach((lineRef) => {
let index0ActPrice;
groupedByLineRef[lineRef].forEach((line, index) => {
//Let the first one keep it
if (index === 0) {
index0ActPrice = line.act_price;
return;
}
//Web Est seems to have additional costs with UNQ_SEQ 0. Keep them all?
if (line.unq_seq === 0) return;
if (index0ActPrice !== line.act_price) {
// line.notes += ` | Price override.`;
return;
}
const indexInEstData = estData.joblines.data.findIndex((l) => l.unq_seq === line.unq_seq);
//estData.joblines.data[indexInEstData].notes +=
// ` | Act Price delete. (prev act price = ${estData.joblines.data[indexInEstData].act_price})`;
estData.joblines.data[indexInEstData].act_price = 0;
estData.joblines.data[indexInEstData].db_price = 0;
});
});
}
});
// InstanceRenderManager({
// executeFunction: true,
// args: [],
// promanager: () => {
// const groupedByLineRef = _.groupBy(estData.joblines.data, "line_ref");
// Object.keys(groupedByLineRef).forEach((lineRef) => {
// let index0ActPrice;
// groupedByLineRef[lineRef].forEach((line, index) => {
// //Let the first one keep it
// if (index === 0) {
// index0ActPrice = line.act_price;
// return;
// }
// //Web Est seems to have additional costs with UNQ_SEQ 0. Keep them all?
// if (line.unq_seq === 0) return;
// if (index0ActPrice !== line.act_price) {
// // line.notes += ` | Price override.`;
// return;
// }
// const indexInEstData = estData.joblines.data.findIndex((l) => l.unq_seq === line.unq_seq);
// //estData.joblines.data[indexInEstData].notes +=
// // ` | Act Price delete. (prev act price = ${estData.joblines.data[indexInEstData].act_price})`;
// estData.joblines.data[indexInEstData].act_price = 0;
// estData.joblines.data[indexInEstData].db_price = 0;
// });
// });
// }
// });
InstanceRenderManager({
executeFunction: true,

1
client/src/components/loading-spinner/loading-spinner.component.jsx
View File

@@ -12,7 +12,6 @@ export default function LoadingSpinner({ loading = true, message, ...props }) {
position: "relative",
alignContent: "center"
}}
// TODO: Client Update - if anything funky happens check this out
{...(props.children ? { tip: message ? message : null } : {})}
delay={200}
// TODO: Client Update - tip only works when there are actually children, and this component is used in a lot of places where there are no children

1
client/src/components/shop-info/shop-info.rbac.component.jsx
View File

@@ -26,7 +26,6 @@ export function ShopInfoRbacComponent({ form, bodyshop }) {
names: ["Simple_Inventory"],
splitKey: bodyshop && bodyshop.imexshopid
});
//TODO:AIO Ensure that there are no duplicates here, it seems like there may be.
return (
<RbacWrapper action="shop:rbac">
<LayoutFormRow>

2
client/src/index.jsx
View File

@@ -31,7 +31,7 @@ if (import.meta.env.PROD) {
Sentry.init({
dsn: InstanceRenderManager({
imex: "https://fd7e89369b6b4bdc9c6c4c9f22fa4ee4@o492140.ingest.sentry.io/5651027",
rome: "https://a6acc91c073e414196014b8484627a61@o492140.ingest.sentry.io/4504561071161344" //TODO:AIO Add in the sentry tracker for proman.
rome: "https://a6acc91c073e414196014b8484627a61@o492140.ingest.sentry.io/4504561071161344"
}),
ignoreErrors: [

2
client/src/pages/jobs-detail/jobs-detail.page.component.jsx
View File

@@ -158,7 +158,7 @@ export function JobsDetailPage({
(acc, val) => {
acc[val] = {
...job.parts_tax_rates[val],
...values.parts_tax_rates?.[val] //TODO:AIO Verify that these still save for Rome Online with this null coalescing.
...values.parts_tax_rates?.[val]
};
return acc;
},

1
client/src/redux/store.js
View File

@@ -30,7 +30,6 @@ export const store = configureStore({
middleware: (getDefaultMiddleware) =>
getDefaultMiddleware({
serializableCheck: false,
// TODO: (Note) This is a production board change
immutableCheck: false
}).concat(middlewares),
// middleware: middlewares,