Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

jooby raml ignore class hidden properties #753

Merged
merged 1 commit into from
May 15, 2017

Conversation

ajcamilo
Copy link
Contributor

@ajcamilo ajcamilo commented May 2, 2017

Jooby RamlType ignore properties starting with _, for example _ebeanServer.

This resolves the issue #718

@ajcamilo
Copy link
Contributor Author

ajcamilo commented May 8, 2017

@jknack if you want I can try to make this "ignore" optional through config.

@jknack jknack mentioned this pull request May 8, 2017
@jknack
Copy link
Member

jknack commented May 8, 2017

@ajcamilo Thank you for your pull and just link from general issue to this particular problem.

We need to do something similar for Swagger (guess).

@ajcamilo
Copy link
Contributor Author

ajcamilo commented May 9, 2017

I tried swagger module, and it works fine. The problem was just with the RAML module.

@ajcamilo
Copy link
Contributor Author

@jknack are you delaying this on purpose? Because of the refactor the spec modules?

I'm asking this because I'm using my forked version of jooby because of this PR and another that I will make because of POJOs with UUIDs. And would like to use the official "jooby".

@jknack jknack added this to the 1.1.2 milestone May 15, 2017
@jknack jknack merged commit 3714c9e into jooby-project:master May 15, 2017
@jknack
Copy link
Member

jknack commented May 15, 2017

Merged it. Going to release 1.1.2 by the end of this week (or early next).

Thank you, don't forget to star/follow the project

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

Successfully merging this pull request may close these issues.

2 participants