Closed ksilz closed 1 year ago
I'm running macOS Ventura 13.2 on an M1 Max 2021 MacBook Pro 16 with the Temurin OpenJDK 17.0.5 and Docker Desktop 4.16.2.
Thanks for reporting.
I'll have a look
https://github.com/redhat-developer-demos/quarkus-petclinic/pull/8 should take care of it
After talking with @mkouba, my fix should not be needed because Qute already provides such an extension method. However we are trying to see why you are getting the error.
@geoand Please wait while I install the tooling so I can download the PR to my local repo.
👍🏼
This might an M1 issue, we are waiting to confirm
Yes, the fix is not correct and should not be merged until we know what the real problem is.
Right, I have converted the PR to draft
But @mkouba we will likely need the fix until we have a Quarkus version that fixes the issue
@gastaldi ran this on his M1 and experience the same issue. So it is M1 specific and we'll have to fix it. But in the meantime @mkouba I propose we merge #8
@geoand Yep, works for me with #8. Thank you!#8
One question: This is how the application looks like this on my Mac (Safari & Firefox). Could it be that the nav bar button doesn't show properly?
@ksilz thanks for checking!
As for the UI, I am totally the wrong person to be able to answer that :) My UI skills is totally non-existent :(, but I can say that it did work properly for me on Chrome on a Linux box.
But in the meantime @mkouba I propose we merge #8
Feel free to merge #8 but (1) pls add some comment to the Temporals
class and (2) file a new issue to revert this commit once we find/fix the real cause.
I added some comments to the code.
The latest commit on
master
gives me this error when runningmvn clean compile quarkus:dev
: