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

fix lists usages #23

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

fix lists usages #23

wants to merge 2 commits into from

Conversation

ZRunner
Copy link

@ZRunner ZRunner commented Oct 21, 2020

Fix issue #22

@danhper
Copy link
Owner

danhper commented Nov 9, 2020

Hi and thank you for taking the time to send a PR.
Could give me a bit more background about the use case and why the normal dictionary structure is not sufficient, please?
Thanks a lot!

@ZRunner
Copy link
Author

ZRunner commented Nov 10, 2020

Hey hello
As I explained in my issue #22, supporting lists (or arrays, as you want) can be really helpful for values which don't require any key paring, like the list of months for example. Doing t(f"month.{i}") seems a bit less natural to me than t("month")[i], and the YAML syntax is way nicer (imo) when you don't have to specify values indexes.
Months list is an example, but I'm sure you may find a tons of other possible usages... basically anything which can be represented as an ordered list.

@MarcoG3
Copy link

MarcoG3 commented Nov 15, 2020

Thanks @ZRunner, I also struggled with lists and your PR helped me. This functionality is a must-have for my current project!

@sralloza
Copy link

Is this going to be implemented at the end?

@ZRunner
Copy link
Author

ZRunner commented Feb 16, 2021

Is this going to be implemented at the end?

I really hope so, but the repo seems dead, which is kind of sad for a such useful lib. I'm thinking of eventually doing a fork later ^^

@harveyives
Copy link

Has there been any movement on this? This module is by far the best solution I've found, yet the list support is a dealbreaker for me unfortunately.

My use-case: I have a list of topics that are selected at random by a bot, it would be great to get these into an i18n friendly format that could be read in by python.

Is there any work I can do to move this along?

@ZRunner
Copy link
Author

ZRunner commented Apr 13, 2021

Well I guess for now you can just use my fork (https://github.com/ZRunner/python-i18n), it's the best we can do so far, until @danhper comes back...

(fyi you can install a package from git with the command pip install git+https://github.com/...)

@Javinator9889
Copy link

It would be awesome if this feature gets implemented :)

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.

6 participants