Skip to content
This repository was archived by the owner on Mar 19, 2025. It is now read-only.

included messages.properties file in app config #25

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nate-getch
Copy link

Need spring.messages.basename defined in app config so it can be accessed from the view templates.

@Raysmond
Copy link
Owner

Raysmond commented Feb 11, 2018

Good! It's better to have a complete way to do i18n stuff. I'll consider it later.

Back to your commit, it seems the spring.messages.basename config path is not right, since it already has a parent path spring

@nate-getch
Copy link
Author

nate-getch commented Feb 14, 2018

If you are open to collaboration and merging my code, I am interested to work on the i18n feature.

Regarding my commit, I have tested the thymeleaf template pages reading the i18n/messages dir properly.

@Raysmond
Copy link
Owner

Yes, you're welcome to contribute. Can you commit a simple example, including the message property files and usages in thymeleaf? So I can review them and see if it's ok to merge.

# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants