-
Notifications
You must be signed in to change notification settings - Fork 2.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
AdminMenuNavigationProvidersCoordinator issue in main branch #15246
Comments
@MikeAlhayek I think you did some changes recently on the navigations APIs |
@sebastienros is this our fist issue after merging STJ PR? |
How is this related to STJ? |
Lines 60 to 75 in 6baa536
|
Because the admin nodes are stored in a document that has to be serialized. I yet had to test it, but you can put a break point in the adminMenuService and see if the nodes mapped to the objects are as expected or not |
@Skrypt in your project, do you have a custom implementation of node type? If so, you'll need to register it like this
|
Nah, I was working either with your ElasticSearch branch or main branch. |
Thanks for the feedback. I added a fix to this issue in #15261 |
This still happens on the latest
|
@sebastienros the last fix seems to only have worked with existing site but not with new sites. I think this is a serialization problem. Can you look at it when you have few mins? Look at the JSON from a site that was serialized using Newtonsoft
Here is one that was creating using STJ "new site"
I am guessing one or both of these is invalid? OrchardCore/src/OrchardCore/OrchardCore.AdminMenu.Abstractions/AdminMenuExtensions.cs Lines 42 to 43 in f8a086b
|
Hi @Skrypt , Does your application still have this problem? My application keeps outputting the same log, even if I use the latest preview package |
Well, solved , I need to clear out the Redis cache.😒 |
I'm getting a lot of these in logs.
The text was updated successfully, but these errors were encountered: