Fix #46: Add new Dev UI

[melloware]

Fix #46: Add new Dev UI

[phillip-kruger]

So the only reason for the custom card is the logo ?

[melloware]

So the only reason for the custom card is the logo ?

Yes pretty much and to shorten the description.

[phillip-kruger]

ok cool, make sense. I think we should add logo/icon to the yaml, but if you need a custom card anyway, then this is good. From my p.o.v this is ok to merge, but I'll let @ia3andy do the final review

[phillip-kruger]

LGTM.

[melloware]

Yes I think we need the logo in the YAML but wasn't sure I would be able to submit a PR that allows a new property in the YAML file? quarkusio/quarkus#31785

[phillip-kruger]

I'll start a discussion and see if we can get it done. For now this is good. Thanks :)

[ia3andy]

@melloware CI is failing

[melloware]

@ia3andy Yeah everything works in 3.0.0.Alpha4 but as soon as you change to Alpha5 or Alpha6 a unit test fails with Expected status code <404> but was <405> without any code changes so I am not sure if something changed in Vert.X or not but didn't want to just change the test to be a 405?

[melloware]

OK I fixed the 404 to 405 but now other tests are failing downloading the NPM binaries? Not sure if its github action related or not?

[melloware]

running locally in Windows I get this failure.

Caused by: java.lang.RuntimeException: Error in Quinoa while running package manager install command: pnpm.cmd install
        at io.quarkiverse.quinoa.deployment.packagemanager.PackageManager.install(PackageManager.java:47)
        at io.quarkiverse.quinoa.deployment.QuinoaProcessor.prepareQuinoaDirectory(QuinoaProcessor.java:108)
        at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
        at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.base/java.lang.reflect.Method.invoke(Method.java:568)
        at io.quarkus.deployment.ExtensionLoader$3.execute(ExtensionLoader.java:909)
        at io.quarkus.builder.BuildContext.run(BuildContext.java:282)
        at org.jboss.threads.ContextHandler$1.runWith(ContextHandler.java:18)
        at org.jboss.threads.EnhancedQueueExecutor$Task.run(EnhancedQueueExecutor.java:2513)
        at org.jboss.threads.EnhancedQueueExecutor$ThreadBody.run(EnhancedQueueExecutor.java:1538)
        at java.base/java.lang.Thread.run(Thread.java:833)
        at org.jboss.threads.JBossThread.run(JBossThread.java:501)

[ia3andy]

@melloware sorry, the 405 bug was not your doing, I created an issue for it: quarkusio/quarkus#31928

[melloware]

Thank you! I will revert my 404 to 405 change.

[melloware]

@ia3andy where did we land on this? I couldn't follow the argument about the 404 vs 405 😄

[ia3andy]

@ia3andy where did we land on this? I couldn't follow the argument about the 404 vs 405 😄

#278 added the fix to the bump PR, waiting for CI

[ia3andy]

@melloware the CI is still failing any idea why?

[melloware]

I am investigating...

[melloware]

OK all passing now!

[ia3andy]

@melloware all is good, the only weird thing is when clicking on items, it leads to an empty page..

[melloware]

Yep open bug I reported to Phillip quarkusio/quarkus#31786

[phillip-kruger]

I'll fix it soon

[ia3andy]

Thanks @melloware for those awesome contrib!