Skip to content

Conversation

@garydgregory
Copy link
Member

The commit is only for new APIs before I dig into an implementation.

Before I proceed I'd like to get feedback on how this looks to the Juneau developers.

The commit is only for new APIs before I dig into an implementation.
No change to the generated HTML yet.
@jamesbognar
Copy link
Contributor

@garydgregory - Are you still interested in implementing this? I'm close to doing a preview release for 9.0. It would be good to get this into 9.0.0.

Note that there have been changes to the way serializers are configured (no more properties like HTMLDOC_xxx and instead just builders now). So unfortunately it's not a clean merge to get your current changes into the code.

@garydgregory
Copy link
Member Author

Hi @jamesbognar
In theory yes, but in practice I have my FOSS time pretty occupied ATM with Apache Commons, so I have to back away for now. I should be available for testing an RC to see if we have any compatibility issues with our product though.

@jamesbognar
Copy link
Contributor

Sounds good. If you want, I can sort out the conflicts and merge the existing changes. Otherwise I'll just leave this PR open for now?

@garydgregory
Copy link
Member Author

@jamesbognar Feel free to merge or do as you best see fit. I apologize for not taking the time to dive back in ATM.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants