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

How many placeholders in defaultlang.json? #183

Closed
Blueberryy opened this issue Jun 29, 2023 · 6 comments · Fixed by #186
Closed

How many placeholders in defaultlang.json? #183

Blueberryy opened this issue Jun 29, 2023 · 6 comments · Fixed by #186
Labels
Enhancement New feature or change request Question Further information is requested

Comments

@Blueberryy
Copy link
Contributor

Can i get full list of placeholders? I only indicated some of them: numberOfComments, waittime, availablenow, untilStr, numberOfVotes and failedamount.

@Blueberryy Blueberryy added the Question Further information is requested label Jun 29, 2023
@3urobeat
Copy link
Owner

3urobeat commented Jun 29, 2023

Hey, thanks for opening an issue!
Good question! This is something I should probably elaborate better in the customlang documentation wiki page.
It could also be useful to indicate them in a different format, for example using the JS syntax ${name} but that would require updating the customlang.json again. I have to think about that in the future.

I quickly went through my code searching for lang\..+\.replace in regex mode, these are all I could find:

  • cmdprefix
  • profileid
  • remainingcooldown
  • waittime
  • availablenow
  • numberOfComments
  • failedamount
  • untilStr
  • thisattempt
  • maxattempt
  • successAmount
  • totalAmount
  • numberOfFavs
  • estimatedtime
  • maxOwnerComments
  • maxComments
  • pingtime
  • groupid
  • requesttime
  • amount
  • value
  • targetkey
  • oldvalue
  • newvalue
  • branchname
  • numberOfVotes
  • maxRequestAmount
  • cmdusage
  • commentcmdusage
  • forceFriendlistSpaceTime
  • unfriendtime

@3urobeat
Copy link
Owner

I hope I didn't miss anything. I have to admit that the current system of "just words that look kinda different inside the string" sucks and I'm not happy with it.
The problem is also that these are not always really unique, meaning the word "value" I listed above could also be a word that should be translated in another string.

If you want to wait before translating I'm going to rework the variable-in-string system and let you know here

@Blueberryy
Copy link
Contributor Author

Hey, thanks for opening an issue! Good question! This is something I should probably elaborate better in the customlang documentation wiki page. It could also be useful to indicate them in a different format, for example using the JS syntax ${name} but that would require updating the customlang.json again. I have to think about that in the future.

I quickly went through my code searching for lang\..+\.replace in regex mode, these are all I could find:

  • cmdprefix
  • profileid
  • remainingcooldown
  • waittime
  • availablenow
  • numberOfComments
  • failedamount
  • untilStr
  • thisattempt
  • maxattempt
  • successAmount
  • totalAmount
  • numberOfFavs
  • estimatedtime
  • maxOwnerComments
  • maxComments
  • pingtime
  • groupid
  • requesttime
  • amount
  • value
  • targetkey
  • oldvalue
  • newvalue
  • branchname
  • numberOfVotes
  • maxRequestAmount
  • cmdusage
  • commentcmdusage
  • forceFriendlistSpaceTime
  • unfriendtime

thank you for the info. I just made Russian translation and i will review my text now

@Blueberryy
Copy link
Contributor Author

@3urobeat
Copy link
Owner

@HerrEurobeat can you review if i missed something? https://github.com/Blueberryy/steam-comment-service-bot/blob/master/src/data/lang/russian.json

Should be fine. Although you need to make sure numberOfComments & numberOfVotes for example in commentprocessstarted & voteprocessstarted aren't capitalized as the replacing is case sensitive.

@3urobeat
Copy link
Owner

If you want you can open a PR to merge your translation into this repository and I'll update it later when I've improved the language system.
Thanks for making one in the first place

@3urobeat 3urobeat added the Enhancement New feature or change request label Jun 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or change request Question Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants