Add DataMapping annotation and new configuration #34
Conversation
roq-data/deployment/src/main/java/io/quarkiverse/roq/data/deployment/RoqDataConfig.java
Show resolved
Hide resolved
mcruzdev
force-pushed
the
issue-19
branch
2 times, most recently
from
e9046b3
to
a84021d
Compare
|
I will squash it after because I mentioned a commit in Zulip. |
| .done()); | ||
| } else if (roqDataMapping.getData() instanceof JsonArray array) { | ||
|
|
||
| if (roqDataMapping.isRecord()) { |
|
@mcruzdev is it ready for review? |
roq-data/deployment/src/main/java/io/quarkiverse/roq/data/deployment/ReadRoqDataProcessor.java
Outdated
Show resolved
Hide resolved
|
sorry @ia3andy @melloware I'm still on vacation. But now we can use parent mapping. I think you can take a initial looking. |
|
I will give it a try today |
There was a problem hiding this comment.
Great stuff!!! Thanks @mcruzdev you roq!
I am getting ClassNotFound exception on a trial local project. I think you need to use Jandex to locate the class:
https://quarkus.io/guides/writing-extensions#scanning-deployments-using-jandex
Could you also add the doc section about data (just add a new page with an index like we do on web-bundler or quinoa, we'll add the rest of the content late)? -- if we don't start it it will be too much to catch up..
|
|
|
@mcruzdev add the doc in another PR ;-) |
Add DataMapping annotation and new configuration
Fixes #19