hilla icon indicating copy to clipboard operation
hilla copied to clipboard

Rename Flux, Pageable and other classes that are used for mapping data

Open Artur- opened this issue 2 years ago • 4 comments

Is your feature request related to a problem? Please describe. When looking for the import for the Flux class, I do not want to see dev.hilla.runtime.transformtypes.Flux, only reactor.core.publisher.Flux

Describe the solution you'd like Rename it to FooBar

Artur- avatar Sep 23 '23 16:09 Artur-

I suspect that, now that we use Jackson to parse Java, the transfer types are not needed anymore, as those types are parsed correctly. Let's investigate.

cromoteca avatar Sep 26 '23 11:09 cromoteca

Another possible approach might be to make the types package private, assuming that can be made to work together with Jackson.

Legioth avatar Feb 27 '25 08:02 Legioth

We could combine several parser modules into a single one if that helps with this issue.

platosha avatar Feb 27 '25 12:02 platosha

A simpler solution would be to rename the mapped type to an arbitrary name, and instead of generating it in TypeScript, generate an import for Flux from a Hilla npm package for that type.

platosha avatar Feb 27 '25 12:02 platosha

Now that the @FromModule was introduced, what remains to do here is finding a replacement for using public empty records for representing these special types during generation.

platosha avatar Jul 22 '25 11:07 platosha