mcullifer / patchhub Goto Github PK

DomainToResponseMapper.cs

Currently ParsingService removes all instances of bold from BBCode to Markdown conversion. Some posts try to bold the list items and the parsing is not account for that. Might need to add condition to figure out if it's inside a list and remove those instances but keep others?

Parsing bbcode to markdown should be done in the infrastructure layer. Instead of directly using steam api responses, create Domain models that are ready to be used on the UI layer in the format that it needs.

This should be part of Mapping. When steam api response object is mapped to Domain object, perform the parsing from bbcode to markdown. This way once database is set up the UI layer in unaffected by different apis and instead relies only on the Domain models.

Could also be beneficial to add Domain layer since number of domain objects is going to grow.

Need I say more?