Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

Catalog Database Seeding Issue #1350

Closed
pavanarya opened this issue Jun 26, 2020 · 2 comments
Closed

Catalog Database Seeding Issue #1350

pavanarya opened this issue Jun 26, 2020 · 2 comments

Comments

@pavanarya
Copy link

pavanarya commented Jun 26, 2020

After getting the latest from this project, when I am trying to run we are getting an error when seeding Catalog database

Exception: does not contain required header picturefilename

Root cause:
In the file src\Services\Catalog\Catalog.API\Infrastructure\CatalogContextSeed.cs

In-Line 184
string[] requiredHeaders = { "catalogtypename", "catalogbrandname", "description", "name", "price", "pictureFileName" }; string[] optionalheaders = { "availablestock", "restockthreshold", "maxstockthreshold", "onreorder" }; csvheaders = GetHeaders(csvFileCatalogItems, requiredHeaders, optionalheaders );

For required headers, we are passing "pictureFileName" but in GetHeaders() when reading csv headers, we are reading headers as lowercase.

Thanks,
Pavan Kumar Aryasomayajulu

@sughosneo
Copy link
Contributor

sughosneo commented Oct 1, 2020

Hi @pavanarya , thank you for pointing that out. Yes, I am able to see similar behavior but that's not affecting the seeding of the CatalogItem table as it's fetching the information from GetPreconfiguredItems() method.

Could you please submit a new PR with the proposed changes ? Please refer comment

@sughosneo
Copy link
Contributor

This has been addressed in #1520
So closing this issue. Please feel free to reopen if you have any further question.

Thank you.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants