afs1jes / Clarins-US-IR-Migration

0 stars 0 forks source link

Load Finance - Ecomm #83

Open AVHuntley opened 1 year ago

AVHuntley commented 1 year ago

Updated SQL grid on FlipOnLoad sheet. Added 'dummy' column to the left of the grid that allows AccountNumber formula to get copied down correctly.

New SQL query: SELECT '' AS AccountNumber, AccountsName, AccountType FROM [Dimension].[Accounts] where AccountType = '-1'

RobSarajian commented 1 year ago

Data Load Report Jan 2022 ECOM Transactions (1)_xlsx 2023-03-31 20-59-40.xlsx

Although it says it loaded in Jan 2022, there is no data showing in Dev.

AVHuntley commented 1 year ago

I was able to find this data in an ad hoc grid, I believe. Here's a workbook that shows my investigation. I've highlighted in yellow regions to compare.

Note the dimensions used in the ad hoc. I was stymied a little in my first search attempt by the fact that many of the dimension .NAMEs in the load tab are not in fact associated with the member whose map value is being loaded to. That may or may not be an error, since A3 only references .NAMEs when it can't find a MAP match. The member names in the grid are those that match the MAP values on the load tab of the load report.

Data Load Investigation (Load Finance - Ecomm).xlsx

Let me know if I can help explain better.

RobSarajian commented 1 year ago

I think the issue is the Load mapping table in the template where it is truncating the department and brand. For example it is treating 000 as 0 or 001 as 1. I think this is the case with all of the load Finance templates.

Rob

From: AVHuntley @.> Sent: Friday, March 31, 2023 7:06 PM To: afs1jes/Clarins-US-IR-Migration @.> Cc: SARAJIAN, Robert @.>; Comment @.> Subject: Re: [afs1jes/Clarins-US-IR-Migration] Load Finance - Ecomm (Issue #83)


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


I was able to find this data in an ad hoc grid, I believe. Here's a workbook that shows my investigation. I've highlighted in yellow regions to compare.

Note the dimensions used in the ad hoc. I was stymied a little in my first search attempt by the fact that many of the dimension .NAMEs in the load tab are not in fact associated with the member whose map value is being loaded to. That may or may not be an error, since A3 only references .NAMEs when it can't find a MAP match. The member names in the grid are those that match the MAP values on the load tab of the load report.

Data Load Investigation (Load Finance - Ecomm).xlsxhttps://urldefense.com/v3/__https:/github.com/afs1jes/Clarins-US-IR-Migration/files/11127795/Data.Load.Investigation.Load.Finance.-.Ecomm.xlsx__;!!OMzJW1YBdAk!oYMEm-2MBcTsVGeZDQISrrTgFmi1fkOcncAUzTGty9uuwEuqBsKEqt5Yzmy_LNcnaTYn3z--5H3lccvTpRozyZb5518H$

Let me know if I can help explain better.

— Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https:/github.com/afs1jes/Clarins-US-IR-Migration/issues/83*issuecomment-1492705537__;Iw!!OMzJW1YBdAk!oYMEm-2MBcTsVGeZDQISrrTgFmi1fkOcncAUzTGty9uuwEuqBsKEqt5Yzmy_LNcnaTYn3z--5H3lccvTpRozyWxA9qtq$, or unsubscribehttps://urldefense.com/v3/__https:/github.com/notifications/unsubscribe-auth/A6WJCL63CUSOXNCPK54GA43W65PNTANCNFSM6AAAAAAVS4JNVI__;!!OMzJW1YBdAk!oYMEm-2MBcTsVGeZDQISrrTgFmi1fkOcncAUzTGty9uuwEuqBsKEqt5Yzmy_LNcnaTYn3z--5H3lccvTpRozySwPZ4ah$. You are receiving this because you commented.Message ID: @.**@.>>

janeslee25 commented 1 year ago

it seems that on the load tab- the department map and brand map get picked up as a number. In the prod system, this doesn't seem to have an issue. We fixed by adding the text function in those fields. seems to have corrected the issue.

AVHuntley commented 1 year ago

Oops, I didn't realize you were looking at this as well. The issue actually stemmed from the SQL table the query was referencing. I updated that this afternoon, probably right around the time you were working on this template!

That fix, in addition to formatting the rows as text within Excel, lets the values come in accurately... except for the first row for some reason. Image

This is something we're looking at, and we'll hopefully have a fix for you soon.