afs1jes / Clarins-US-IR-Migration

0 stars 0 forks source link

Load Finance - AUTO #81

Open AVHuntley opened 1 year ago

RobSarajian commented 1 year ago

Dev vs Prod Feb 2023.xlsx

The load is working ok but the signs are reversed for all Net sales accounts and Financial Income. The differences are from New Accounts that are not in the Dev mapping table. I have attached file showing Dev vs Prod.

AVHuntley commented 1 year ago

I think I've found the issue. Working on a fix now.

Has to do with the sheet called FlipOnLoad. Column B, left-adjacent to the SQL grid, contains a formula in the formatting row that, on prod, gets copied down the grid on build.

Image

(dev on left; prod on right)

RobSarajian commented 1 year ago

OK Thanks – same issue with Load Chain AUTO and probably all of the other loads.

Rob Sarajian

From: AVHuntley @.> Sent: Thursday, March 30, 2023 2:49 PM To: afs1jes/Clarins-US-IR-Migration @.> Cc: SARAJIAN, Robert @.>; Comment @.> Subject: Re: [afs1jes/Clarins-US-IR-Migration] Load Finance - AUTO (Issue #81)


WARNING: External email. Please verify sender before opening attachments or clicking on links.


I think I've found the issue. Working on a fix now.

Has to do with the sheet called FlipOnLoad. Column B, left-adjacent to the SQL grid, contains a formula in the formatting row that, on prod, gets copied down the grid on build.

[Image removed by sender. Image]https://urldefense.com/v3/__https:/user-images.githubusercontent.com/88676502/228934925-f4c3a398-1e4f-4677-bcb5-167082c5d374.png__;!!OMzJW1YBdAk!uPFWNGRdCUdyv5Oxlstp3snlJ_SBsrnOnS9JEECdrQCkvPMtxeIc4EMzqG3UHbjRFvVB8KCkt_p1BNTATH7IvjntKJ0J$

(dev on left; prod on right)

— Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https:/github.com/afs1jes/Clarins-US-IR-Migration/issues/81*issuecomment-1490768707__;Iw!!OMzJW1YBdAk!uPFWNGRdCUdyv5Oxlstp3snlJ_SBsrnOnS9JEECdrQCkvPMtxeIc4EMzqG3UHbjRFvVB8KCkt_p1BNTATH7Ivk3xsWoD$, or unsubscribehttps://urldefense.com/v3/__https:/github.com/notifications/unsubscribe-auth/A6WJCL2GUQ4FMOJSBYBCLF3W6XISRANCNFSM6AAAAAAVS4CM2I__;!!OMzJW1YBdAk!uPFWNGRdCUdyv5Oxlstp3snlJ_SBsrnOnS9JEECdrQCkvPMtxeIc4EMzqG3UHbjRFvVB8KCkt_p1BNTATH7Ivtv0_dd9$. You are receiving this because you commented.Message ID: @.***>

AVHuntley commented 1 year ago

Okay, change made and saved. Had to modify the SQL query to add a 'dummy' row to leave space for the formula to fill. It doesn't affect all records on the load sheet, but I did see at least one +/- flip on a record, so I suspect this was the problem. Try testing again whenever you get a chance and let me know how it goes.

I'll look a the other load sheet right now.

RobSarajian commented 1 year ago

I reloaded both the Finance & Chain AUTO and the results are still the same.

Rob

From: AVHuntley @.> Sent: Thursday, March 30, 2023 3:31 PM To: afs1jes/Clarins-US-IR-Migration @.> Cc: SARAJIAN, Robert @.>; Comment @.> Subject: Re: [afs1jes/Clarins-US-IR-Migration] Load Finance - AUTO (Issue #81)


WARNING: External email. Please verify sender before opening attachments or clicking on links.


Okay, change made and saved. Had to modify the SQL query to add a 'dummy' row to leave space for the formula to fill. It doesn't affect all records on the load sheet, but I did see at least one +/- flip on a record, so I suspect this was the problem. Try testing again whenever you get a chance and let me know how it goes.

I'll look a the other load sheet right now.

— Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https:/github.com/afs1jes/Clarins-US-IR-Migration/issues/81*issuecomment-1490827780__;Iw!!OMzJW1YBdAk!u9SoQM7DHJf3X0iKEa8ZxkgLYHWXeGZGHSprwtEm4uzkUMaF3F6oXOEWrXvQModTjCumgs-T2WqDUYUf1m1yFbI8dYnJ$, or unsubscribehttps://urldefense.com/v3/__https:/github.com/notifications/unsubscribe-auth/A6WJCLZ237ZTVEQBPKZX63LW6XNOHANCNFSM6AAAAAAVS4CM2I__;!!OMzJW1YBdAk!u9SoQM7DHJf3X0iKEa8ZxkgLYHWXeGZGHSprwtEm4uzkUMaF3F6oXOEWrXvQModTjCumgs-T2WqDUYUf1m1yFdbhCfSD$. You are receiving this because you commented.Message ID: @.***>

AVHuntley commented 1 year ago

In changing the above, I'd accidentally changed the boundaries of the named range FlipAccountsOnLoad. Fixing that range address fixed the issue in the formula.

afs1jes commented 1 year ago

We've tested the account flipping calculation using sample data. This should be ready for loading data.