afs1jes / Clarins-US-IR-Migration

0 stars 0 forks source link

Load SISO Item Level #84

Open AVHuntley opened 1 year ago

AVHuntley commented 1 year ago

The last sheet, SqlScriptsAfter, has a SQL script in A1. Not sure if that affects functionality.

RobSarajian commented 1 year ago

Data Load Report SISO by item and retailer-en-us_xlsx 2023-05-05 13-56-58.xlsx

File did not load

AVHuntley commented 1 year ago

Seems like there was an issue with the formula being used on the Load sheet for Pyramid.MAP. It wasn't an issue on the prod system, but an invisible difference in the way the data returns on the SQL grids in Dev, the value of a seemingly empty cell was returning as "" instead of "0" (so the formula was returning "" instead of "NONE").

I altered the formula by wrapping that condition in a VALUE() function, which seems to take care of this.

Let us know if this seems to fix the issue for you as well.

disregard. That fix didn't work after all. Still thinking about it.

AVHuntley commented 1 year ago

This change works, I think. We add an additional check to the IF statement in the formula. Now it checks for both "0" and "" values. If either is found, Pyramid.MAP = "NONE"

=IFERROR(IF( AND( VLOOKUP(G2,Retailers!DataSet1,3,FALSE)<>0, VLOOKUP(G2,Retailers!DataSet1,3,FALSE)<>"" ), VLOOKUP(G2,Retailers!DataSet1,3,FALSE),"NONE"),"NONE")