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 weirdness with datacore rendering HAS_SKIN_COLOR races as default color #561

Merged
merged 1 commit into from Mar 11, 2015

Conversation

tigercat2000
Copy link
Contributor

For some reason in datacore, it was checking for "HAS_SKIN_TONE" and then "HAS_SKIN_COLOR" which does not work, as no race has both "HAS_SKIN_TONE" and "HAS_SKIN_COLOR".

Edit: To specify, the datacore handles things such as ID images. @Spacemanspark

@mvanalphen mvanalphen added the Fix This PR will fix an issue in the game label Mar 11, 2015
@Spacemanspark
Copy link
Contributor

Interesting.
Well, thanks for the fix.

ZomgPonies added a commit that referenced this pull request Mar 11, 2015
Fix weirdness with datacore rendering HAS_SKIN_COLOR races as default color
@ZomgPonies ZomgPonies merged commit 3eed0f6 into ParadiseSS13:master Mar 11, 2015
@tigercat2000 tigercat2000 deleted the TajIDfix branch March 11, 2015 20:48
Zigamus pushed a commit to Zigamus/Paradise that referenced this pull request Jan 16, 2022
* Fix: Avoid sleeping the ticker

* Refactor: Use waitfor

Co-authored-by: Simon Ogorodnik <simon.ogorodnik@jetbrains.com>
mattgroy pushed a commit to mattgroy/Paradise that referenced this pull request Nov 10, 2023
<!-- Пишите **НИЖЕ** заголовков и **ВЫШЕ** комментариев, иначе что то
может пойти не так. -->
<!-- Вы можете прочитать Contributing.MD, если хотите узнать больше. -->

## Что этот PR делает
Чиню новую еду.
Возвращаю новой еде спрайты.

<!-- Вкратце опишите изменения, которые вносите. -->
<!-- Опишите **все** изменения, так как противное может сказаться на
рассмотрении этого PR'а! -->
<!-- Если вы исправляете Issue, добавьте "Fixes ParadiseSS13#1234" (где 1234 - номер
Issue) где-нибудь в описании PR'а. Это автоматически закроет Issue после
принятия PR'а. -->

## Почему это хорошо для игры
Еду можно приготовить и её видно.

<!-- Опишите, почему, по вашему, следует добавить эти изменения в игру.
-->

## Изображения изменений
Всё в коде и в икондиффе

<!-- Если вы не меняли карту или спрайты, можете опустить эту секцию.
Если хотите, можете вставить видео. -->

## Тестирование
В процессе, но не мной Jokerge
Тестер - Seiko (Кирр Иллов) aka. mrseikore

<!-- Как вы тестировали свой PR, если делали это вовсе? -->

## Changelog

:cl:
fix: Новые пиццы теперь готовятся корректно.
imageadd: Вернул спрайт овощному салату.
/:cl:

<!-- Оба :cl:'а должны быть на месте, что-бы чейнджлог работал! Вы
можете написать свой ник справа от первого :cl:, если хотите. Иначе
будет использован ваш ник на ГитХабе. -->
<!-- Вы можете использовать несколько записей с одинаковым префиксом
(Они используются только для иконки в игре) и удалить ненужные. Помните,
что чейнджлог должен быть понятен обычным игроком. -->
<!-- Если чейнджлог не влияет на игроков(например, это рефактор), вы
можете исключить всю секцию. -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix This PR will fix an issue in the game
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants