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

English Stringtable Update #3051

Merged

Conversation

Tiny-DM
Copy link

@Tiny-DM Tiny-DM commented Nov 15, 2023

What type of PR is this.

  1. Bug
  2. Change
  3. Enhancement

What have you changed and why?

Information:

A lot of changes to addons/core/Stringtable.xml to improve clarity and grammar errors in certain places. Beyond a few specific grammar errors or typos, most of the changes were left to my purview and I've attempted to make them clearer for new players to understand. Let me know about any changes you think are necessary.

Please specify which Issue this PR Resolves.

closes #3046

Please verify the following and ensure all checks are completed.

  1. Have you loaded the mission in LAN host?
  2. Have you loaded the mission on a dedicated server?

Is further testing or are further changes required?

  1. No
  2. Yes (Please provide further detail below.)

How can the changes be tested?

Steps:
All of the "problematic strings"' have been tested (e.g. strings with specific formatting and breaks in them), the only strings left are the strings relating to Garbage Cleaning, members, and FF punishments because those aren't things I can test on a LAN host and I am far too smooth brain to go through the effort of setting up a dedicated server to test basic changes to strings.


Notes:

A3A/addons/core/Stringtable.xml Show resolved Hide resolved
A3A/addons/core/Stringtable.xml Outdated Show resolved Hide resolved
A3A/addons/core/Stringtable.xml Outdated Show resolved Hide resolved
A3A/addons/core/Stringtable.xml Outdated Show resolved Hide resolved
A3A/addons/core/Stringtable.xml Outdated Show resolved Hide resolved
A3A/addons/core/Stringtable.xml Outdated Show resolved Hide resolved
A3A/addons/core/Stringtable.xml Outdated Show resolved Hide resolved
A3A/addons/core/Stringtable.xml Outdated Show resolved Hide resolved
A3A/addons/core/Stringtable.xml Outdated Show resolved Hide resolved
@Tiny-DM
Copy link
Author

Tiny-DM commented Nov 18, 2023

Fixed. Are 860 and 886 just for consistency?

@Bob-Murphy
Copy link

Fixed. Are 860 and 886 just for consistency?

It's fun, known and already translated. No real need to change it.

A3A/addons/core/Stringtable.xml Outdated Show resolved Hide resolved
@Bob-Murphy Bob-Murphy added this to the 3.4.1 milestone Nov 22, 2023
@Bob-Murphy Bob-Murphy merged commit a7892c8 into official-antistasi-community:unstable Nov 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants