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

Improve documentation of Configuration.register() #667

Closed
cowwoc opened this issue Feb 14, 2014 · 1 comment
Closed

Improve documentation of Configuration.register() #667

cowwoc opened this issue Feb 14, 2014 · 1 comment
Labels
Milestone

Comments

@cowwoc
Copy link
Contributor

@cowwoc cowwoc commented Feb 14, 2014

I read the Javadoc and http://www.querydsl.com/static/querydsl/3.3.1/reference/html/ch02s03.html#d0e904 and it's still not clear to me how Configuration.register() converts data from/to the SQL type and the Java type I specify.

I'm guessing that you convert from SQL to Java by passing in the SQL parameters as arguments to the Java constructor of the type I provide, but it's not clear how you go the other way.

Please document this process more explicitly in the Javadoc, or add a link from the Javadoc to a document that explains it in more details.

@timowest timowest added the docs label Feb 14, 2014
timowest added a commit that referenced this issue Mar 20, 2014
timowest added a commit that referenced this issue Mar 20, 2014
@timowest timowest added the fixed label Mar 20, 2014
@timowest
Copy link
Member

@timowest timowest commented Mar 24, 2014

Released in 3.3.2

@timowest timowest closed this Mar 24, 2014
@timowest timowest added this to the 3.3.2 milestone Apr 13, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants