Skip to content

Commit 745e562

Browse files
committed
Mark IPageTopicViewModel as obsolete
There's no expected use case for the `IPageTopicViewModel`. We foresee the limited but potential need for e.g. shared view models that will handle navigation, but these can be handled via `INavigableTopicViewModel` and `INavigationTopicViewModel<T>`. Others _may_ need additional details, which are available via `ITopicViewModel`. But we don't expect reusable _layouts_, which is where the additional requirements of e.g. `IPageTopicViewModel` comes in. On an assessment of OnTopic 4.x implementations, it doesn't appear anyone is utilizing `IPageTopicViewModel`, except possibly on model definitions, so it should be safe to remove. Marking it as obsolete.
1 parent 68a4e83 commit 745e562

1 file changed

Lines changed: 2 additions & 0 deletions

File tree

OnTopic/Models/IPageTopicViewModel.cs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
| Client Ignia, LLC
44
| Project Topics Library
55
\=============================================================================================================================*/
6+
using System;
67
using OnTopic.Mapping;
78

89
namespace OnTopic.Models {
@@ -21,6 +22,7 @@ namespace OnTopic.Models {
2122
/// provided via the public interface then it will instead need to be defined in some other way.
2223
/// </para>
2324
/// </remarks>
25+
[Obsolete("The IPageTopicViewModel is no longer utilized.", true)]
2426
public interface IPageTopicViewModel : ITopicViewModel, INavigableTopicViewModel {
2527

2628
/*==========================================================================================================================

0 commit comments

Comments
 (0)