search

afs1jes / Clarins-US-IR-Migration

0 stars 0 forks source link

Load Chain - AUTO #86

Open AVHuntley opened 1 year ago

RobSarajian commented 1 year ago

Same as Load Finance-Auto. Load is Ok but All Net Sales & Financial Income accounts are reversed.

AVHuntley commented 1 year ago

Oops, got my comments reversed, but the comment I made on 'Load Finance - AUTO' should apply to this template. And I made the same change to both.

Grid should now look like this:

Image

I'm keeping 'disable rebuild' selected, since rebuilding sql grids client-side can sometimes cause issues unless we rewrite the queries to avoid the issue. Not a big deal on load templates though, since they're not really made to be rebuilt.

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.

RobSarajian commented 1 year ago

Data Load Report ClarinsECOM_CNA_LBC_PL_xlsx 2023-05-05 18-45-30.xlsx Data Load Report ClarinsUS_PL_xlsx 2023-05-05 18-37-23.xlsx

It says it load complete but did not load. see below ad hoc

Book1.xlsx

AVHuntley commented 1 year ago

On the 'Errors and Warnings' tab, I saw that the issue had to do with invalid column names in two SQL tables. In the new system, this includes any repeated column names, which both of those grids contained. This caused those tables to not build, and, I'm assuming, messed up any formulas that did lookups against those tables.

I fixed the two grids in slightly different ways.

For RetailerMaps, I altered the SQL query to insert a 'dummy' column for the formula to get pasted into on build.

For LoadMappings, I renamed the first column (which contains a formula anyway) [full_GL_code]. This is a valid column name in the table, so it won't throw an error on build. Note that this column in Excel is populated filled with formulas on build, so it won't actually end up showing values from the table. However, the values in that table column seem to be similar to those generated by the on-sheet formula. Maybe it would make more sense to just get the table values instead then? Just a thought.

RobSarajian commented 1 year ago

Load failed - see file

Data Load Report ClarinsUS_PL_xlsx 2023-05-09 15-05-52.xlsx

RobSarajian commented 1 year ago

Load file

ClarinsUS_PL.xlsx

AVHuntley commented 1 year ago

I see on the Summary sheet of that load report that the load was going to the Financial::Finance cube. Is that correct?

Image

RobSarajian commented 1 year ago

Yes I realized it after. I think it’s good now.

Thanks,

Rob

From: AVHuntley @.> Sent: Tuesday, May 9, 2023 12:26 PM To: afs1jes/Clarins-US-IR-Migration @.> Cc: SARAJIAN, Robert @.>; Comment @.> Subject: Re: [afs1jes/Clarins-US-IR-Migration] Load Chain - AUTO (Issue #86)

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

I see on the Summary sheet of that data load that the load was going to the Financial::Finance cube. Is that correct?

[Image]https://urldefense.com/v3/__https:/user-images.githubusercontent.com/88676502/237159329-6c124b28-500b-4fad-b376-f5e3ed9dd445.png__;!!OMzJW1YBdAk!o37yf_nSwiSicYfySsFKgyq2YUZv6zSdE7rpvXpIEcMCUAEUp2P9ctDWIpvqpjOFE4MRnFkme9xeypIMn03NZsUUEFXV$

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