TimGeyssens / UIOMatic

Auto generate an integrated crud UI in Umbraco for a db table based on a petapoco poco (and more)
https://timgeyssens.gitbook.io/ui-o-matic/
Other
67 stars 54 forks source link

Update umbraco 9 #193

Closed marciogoularte closed 2 years ago

marciogoularte commented 3 years ago

Hi,I did an upgrade to umbraco 9. It's working very well. But I couldn't make a pull request. How can I proceed?

matic

TimGeyssens commented 3 years ago

wow sweet @marciogoularte ! well did you make a fork on github? Then you should just be able to create a pull request? https://docs.github.com/en/github/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/creating-a-pull-request

TimGeyssens commented 3 years ago

alternatively you can also send me the files and I can setup a new branch

TimGeyssens commented 3 years ago

@marciogoularte ? Let me know how I can help :)

marciogoularte commented 3 years ago

Hi, i made a pull request. As you may know, Umbraco v9 everything is by DI. We no longer have Umbraco.Core.Composing.Current. UIOMatic uses static classes using Umbraco v8 Current, I had to make a service locator. But I believe that temporary, we can improve this

TimGeyssens commented 3 years ago

Sweeeeet! Thank you so much. Just need to see how I can merge to a new branch. @marciogoularte

TimGeyssens commented 3 years ago

merged to a new branch @marciogoularte , now just need to figure out why the automated build doesn't work

sbosell commented 3 years ago

Looking forward to seeing this released!

TimGeyssens commented 2 years ago

@marciogoularte mananaged to get the appveyor build working https://www.nuget.org/packages/Nibble.Umbraco.UIOMatic/

TimGeyssens commented 2 years ago

not sure if the nuget package needs to change...

marciogoularte commented 2 years ago

Hi @TimGeyssens . I'm not sure either

TimGeyssens commented 2 years ago

yeah need to look into it, doesn't seem to do the correct stuff...

harvzor commented 2 years ago

I'm trying it out now. I had the issue:

System.InvalidOperationException: Unable to resolve service for type 'UIOMatic.IUIOMaticHelper' while attempting to activate 'UIOMatic.Web.Controllers.UIOMaticTreeController'.

Adding:

services.AddSingleton<IUIOMaticHelper, UIOMaticHelper>();

In my Startup.ConfigureServices method seemed to help.

Update

I'm trying to upgrade to Umbraco v9 but the site was having a strange issue:

No controller/action found by name Render.Index

My project structure is like:

Seems that Nibble.Umbraco.UIOMatic.Core depends on Umbraco.Cms.Core ≥ 9.0.0 and Umbraco.Cms ≥ 9.0.0, but I doubt it should be using Umbraco.Cms?

I don't want my Core project to have a sub-dependency of Umbraco.Cms as it adds files like ~/umbraco/UmbracoWebsite/NoNodes.cshtml which I think messes with the build.

My site doesn't really need UIOMatic that much so I'm disabling it now. Not looking for a solution anytime soon, thought I'd just add my thoughts.

TimGeyssens commented 2 years ago

thanks @Harvzor , yeah the nuget package needs some work, thanks for letting me know

davidsandstrom commented 2 years ago

What do i have to do to get this working? Registered .AddUIOMatic() in startup but now i get a null reference exception in /umbraco/ServerVariables: "NullReferenceException: Object reference not set to an instance of an object. UIOMatic.Web.UIOMaticServerVariablesHandler.Handle(ServerVariablesParsingNotification notification)" in backoffice and a bunch of errors from angular.

TimGeyssens commented 2 years ago

@marciogoularte any pointers?

dataworking commented 2 years ago

What do i have to do to get this working? Registered .AddUIOMatic() in startup but now i get a null reference exception in /umbraco/ServerVariables: "NullReferenceException: Object reference not set to an instance of an object. UIOMatic.Web.UIOMaticServerVariablesHandler.Handle(ServerVariablesParsingNotification notification)" in backoffice and a bunch of errors from angular.

That will be missing config elements in appsettings.json - I commented in more detail in https://github.com/TimGeyssens/UIOMatic/issues/197

TimGeyssens commented 2 years ago

version 4.1.0 adds default config