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

application.properties in README.md instead of jasperreports.properties #77

Closed
dvht opened this issue Apr 14, 2021 · 3 comments
Closed
Assignees

Comments

@dvht
Copy link

dvht commented Apr 14, 2021

Hi Dietmar,

I helped my colleages today with switching from JDBC to JNDI, but some of the instructions in README.md are misleading.
"configuration" chapter says "jasperreports.properties file", but application.properties is meant.

Regards,
Dirk

@daust
Copy link
Owner

daust commented Apr 14, 2021

Yeah, I should update the instructions. I hardly use the JNDI integration any more.

Why did you switch and which version are you using, 2.7.0?

@dvht
Copy link
Author

dvht commented May 14, 2021

Hi Dietmar,

Excuse me for ignoring your question so long. I lost track of this. We use JNDI primary for having identical configuration setup over all java web applications. And, besides, the number of connections is limited and connections already made.
I can't remember which team asked the question, so I can't say anymore which version they were using.

Regards,
Dirk

@daust daust self-assigned this Feb 8, 2022
@daust daust added this to the Release v2.8.0 milestone Feb 8, 2022
@daust
Copy link
Owner

daust commented Feb 8, 2022

Sorry, can't find the error any longer, seems fine. Perhaps I have fixed it along the way.

@daust daust closed this as completed Feb 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants