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

refactor: two-character keys for languages. #4215

Merged
merged 12 commits into from
Feb 21, 2024

Conversation

Daeberdir
Copy link
Member

@Daeberdir Daeberdir commented Jan 14, 2024

Описание

Немного рефакторит систему языков:

  • Манипуляция с языками теперь дополнительно может проходить через их ключи, для этого там будет генерироваться статичный список.
  • Все ключи языков переведены на двухсимвольные значения по типу "un" для языка унати.
  • Пофикшен баг того, что можно было "вслух" говорить хайвмайндом, если иметь хайвмайнд дефолтным языком и пытаться ввести вначале сообщения префикс языка, который персонаж не знает.
  • Ещё из важных изменений (или лучше сказать вынужденых) - теперь, чтобы душить радиоканалы эмоутами, нужно вписывать знак восклицания перед каналом связи, а не после него. Пример: "!;зевает."
  • А, ну и ещё теперь можно ввести префикс языка как до префикса радиоканала, так и после.

Ссылка на предложение

https://discord.com/channels/617003227182792704/755125334097133628/1182693322066772038

По непонятным моментам

Коротко. Не везде мы можем использовать названия языков. Единственным хорошим вариантом будет расширение манипуляции языками на их ключи.

@github-actions github-actions bot added Merge Conflict This doesn't seem right and removed Merge Conflict This doesn't seem right labels Feb 6, 2024
@ArtemDerp
Copy link
Contributor

@Vladisvell @BeebBeebBoob @SAADf603 вот бы аппрув увидеть

@github-actions github-actions bot removed the Merge Conflict This doesn't seem right label Feb 13, 2024
@LiquidPotroh
Copy link
Contributor

Ух, вот бы ещё какой нибудь красивый кодер в объединении с не менее красивым лороведом разделил языки по тому, может ли на них говорить только определённая раса из за физеологических особенностей(например язык кидан, у них там движения усиками часть языка), или же его можно выучить и другим(синтаунати). И привязать к майнду все те, что не привязаны к расе. Чтобы например при пересадке такие языки сохранялись. И в обратную сторону, чтобы люди не узнавали синтаунати, если их мозг вдруг очутился в ящере.
Это же так просто будет сделать в свежеотрефакторенном коде, да?)

BeebBeebBoob
BeebBeebBoob previously approved these changes Feb 18, 2024
code/datums/datumvars.dm Outdated Show resolved Hide resolved
code/modules/client/preference/preferences_mysql.dm Outdated Show resolved Hide resolved
Daeberdir and others added 7 commits February 19, 2024 08:53
commit 069e324
Author: Daeberdir <[email protected]>
Date:   Sun Jan 14 21:58:23 2024 +0200

    I said DELETING

commit c8fc540
Author: Daeberdir <[email protected]>
Date:   Sun Jan 14 21:57:53 2024 +0200

    Deleting test lang.

commit e480281
Author: Daeberdir <[email protected]>
Date:   Sun Jan 14 21:55:06 2024 +0200

    Done done done.

commit f5b5324
Author: Daeberdir <[email protected]>
Date:   Sun Jan 14 17:56:40 2024 +0200

    Update borer.dm

commit 0f11f97
Author: Daeberdir <[email protected]>
Date:   Sun Jan 14 17:07:02 2024 +0200

    Sinta'tajr fixes.

commit 5a08055
Author: Daeberdir <[email protected]>
Date:   Sat Jan 13 22:02:25 2024 +0200

    Sinta'tajr

commit 5570c23
Author: Daeberdir <[email protected]>
Date:   Sat Jan 13 07:48:47 2024 +0200

    Done.
Co-authored-by: ArtemDerp <[email protected]>
Co-authored-by: ArtemDerp <[email protected]>
@Dimach Dimach merged commit 68ba68c into ss220-space:master220 Feb 21, 2024
10 checks passed
@Daeberdir Daeberdir deleted the yaziki branch February 22, 2024 07:22
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