Developer forum

Forum » Dynamicweb 10 » Custom Marketplace AddInProvider error in navigation

Custom Marketplace AddInProvider error in navigation

Mario Santos Dynamicweb Employee
Mario Santos
Reply

Hi, 

We have created a custom marketplace AzureArtifactsAddinProvider but we are getting a UI error.

System.NotImplementedException: The method or operation is not implemented.
   at Dynamicweb.Apps.UI.Tree.AppstoreNodePathProvider.GetNavigationNodePathInternal(AddinDataModel model)
   at Dynamicweb.CoreUI.Navigation.NavigationNodePathProvider`1.GetNavigationNodePath(TModel model)
   at Dynamicweb.CoreUI.Navigation.NavigationNodePathProvider`1.GetNavigationNodePath(DataListViewModel`1 listResult)


Every AddinInfo we return sets AddinProvider = typeof(AzureArtifactsAddinProvider).FullName (in ToAddinInfo). When the App Store admin screen builds its navigation tree, it calls this method for each addin model — and since our provider's type name isn't NuGetAddinProvider, LocalAddinProvider, or empty, it falls through to throw new NotImplementedException().

The App Store UI in this Dynamicweb version hard-codes support for only the two built-in providers; a third custom AddinProvider isn't handled by the navigation tree.

https://dev.azure.com/dynamicwebsoftware/Dynamicweb/_git/Dynamicweb10?path=/Dynamicweb.Apps.UI/Tree/AppstoreNodePathProviders.cs&_a=contents&version=GBmaster

Could you please change the navigation provider to allow new addins? Dropping in All Apps would be good.

 

BR Mario


Replies

 
Jeppe Eriksson Agger Dynamicweb Employee
Jeppe Eriksson Agger
Reply

Hi Mario,

This must have been an oversight that unfortunately was left in. I've removed the requirement for the provider to be one of ours, and made any explicitly set provider behave just like the NuGet and Local providers did before.

You can follow along here: https://doc.dynamicweb.dev/documentation/fundamentals/dw10release/releasenotes/workItemInfo.html?workitemid=28947

Can you tell me which version of DW10 you're using? That way I can try to get it merged down.

- Jeppe

 
Mario Santos Dynamicweb Employee
Mario Santos
Reply

Hi Jeppe,

We are currently on R0, but we will likely need this for other projects so if we could add it to current R1 and R2 would be great!

BR Mario

 

You must be logged in to post in the forum