Context namespace is messy, especially the `MarkdownContext` object - Githubissues

David-Rushton / Blog

Data dispatches

MIT License

0 stars 0 forks source link

Context namespace is messy, especially the `MarkdownContext` object #64

Open David-Rushton opened 4 years ago

David-Rushton commented 4 years ago

This API has a number of problems including:

MarkdownContext is comprised of several items stored in just one file
MarkdownContext includes a tag builder. There is a dedicated ContextBuilder.
MarkdownContext is a complex object for consumers to work with (GetDelimitedTags is not part of the Tags object for example).
Variable substitution has to be handled by clients when this should be a shared service ($(article-id), $(article-title), etc).

The API requires a full review and, probably, a refactor.