Skip to content

Conversation

@joaomfrebelo
Copy link

…nce.

Create of PrestaSharpNotAuthorazedException and PrestaSharpNotFoundException.
Create of a FilterFactory class to facilitate the use of GetByFilter.
Create of two method in Functins class to clean strings as in Prestashop Validate:isCatalogName($name) and Validate:isGenericName($name)

…nce.

Create of PrestaSharpNotAuthorazedException and PrestaSharpNotFoundException.
Create of a FilterFactory class to facilitate the use of GetByFilter.
Create of two method in Functins class to clean strings as in Prestashop Validate:isCatalogName($name) and  Validate:isGenericName($name)
@mowcixo
Copy link
Collaborator

mowcixo commented Dec 19, 2018

Hello @joaomfrebelo, thanks for the PR! Do you think you could separate the atomic functionalities in separated PRs? It would be better to review.

Also, with latest changes in master, GenericFactory.cs has conflicts, can you take a look?

@LameuleFR
Copy link
Collaborator

Hi @joaomfrebelo

I know it's a bit late but do u think u can finalize this PR?

Benjamin

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants