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

Lang support YRB(.pres) format #99

Closed
wants to merge 5 commits into from
Closed

Conversation

dmi3y
Copy link

@dmi3y dmi3y commented Aug 28, 2013

@dmi3y
Copy link
Author

dmi3y commented Aug 28, 2013

actually there is still some tests cleanups would be positive, need early feedback.

@caridy
Copy link
Member

caridy commented Aug 28, 2013

We are trying to move away from YRB format (which is a proprietary format btw), which makes me think that this feature should not be part of shifter. An alternative solution will be to create a new cli tool to do pres2json and use that in your build.json as a script to generate those files, or ultimately, to get shifter to use that module when needed.

@dmi3y
Copy link
Author

dmi3y commented Aug 28, 2013

Hi Caridy, thanks for response.

Okay, that makes sense, I think it would be good to have that mentioned into documentation ideally on both intl and shifter sides. Right now the intl documentations makes promise for supporting it into shifter.

The YUI loader expects resource bundles in a specific format. If you use Shifter to build your module, resource bundles in JSON or YRB format will be automatically converted into the format expected by the loader. All you have to do is provide the source files in the src/module/lang/ directory and add the lang keys to the JSON file under src/module/meta/.

Also wondering what's is particular reasons for moving it away? That's seems to be nice feature for projects which are using fmt tag library for jsp.
If that only because proprietary wars? How it could makes people stop using it? And who is actual owner Yahoo or Oracle? Sorry for so many questions :)

@dmi3y
Copy link
Author

dmi3y commented Aug 28, 2013

I'm closing btw

@dmi3y dmi3y closed this Aug 28, 2013
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