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.

Sign up for GitHub

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

Jump to bottom

Новые шаблоны ошибок для мапперов и переводчиков #13438

Merged
merged 26 commits into from
Sep 29, 2024
Merged

Новые шаблоны ошибок для мапперов и переводчиков #13438

merged 26 commits into from
Sep 29, 2024

Conversation

AzzyDreemurr13
Copy link
Contributor

@AzzyDreemurr13 AzzyDreemurr13 commented Aug 6, 2024 •
edited
Loading

Описание изменений

Новые шаблоны по актуальным проблемам. Добавлены шаблоны по мапперским ишуям и перевод ишуям

Можете глянуть как это будет выглядеть:
https://github.com/TauCetiStation-TranslationDepartment/TauCetiClassicNewTemplate/issues/new/choose

Почему и что этот ПР улучшит

Более корректные баг репорты с установкой нужных лейблов

Авторство

Я

Чеинжлог

Не требуется, ибо изменения будут на гитхабе

@TauKitty
Copy link
Contributor

TauKitty commented Aug 6, 2024

Changelog status: 🔘 No changelog

@TauKitty TauKitty requested a review from a team August 6, 2024 06:18
@AzzyDreemurr13 AzzyDreemurr13 requested a review from a team August 6, 2024 06:19
KatoMarika
KatoMarika approved these changes Aug 6, 2024
View reviewed changes
Copy link
Member

@KatoMarika KatoMarika left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Мап деп. Итоги.

Скрины.

image
image
Мне тоже норм.

С учетом того как у нас мертвецы стоят по отделу, думаю можно апрувнуть, кому не понравится - будет воевать потом.

@LudwigVonChesterfield
Copy link
Contributor

эти лейблы не настолько важны чтобы конфузить создающего ишуй мне кажется.

@AzzyDreemurr13
Copy link
Contributor Author

эти лейблы

Если ты про Мап ишуй и транслейшион, то это будет сразу выделять из потока баг репоторов, ибо чаще выпускают ишуи или связанные с картой или связанные с ошибкой в тексте. Поэтому, эти лейблы сами будут добавляться

@TauKitty TauKitty requested a review from a team August 22, 2024 17:11
KIBORG04
KIBORG04 requested changes Sep 2, 2024
View reviewed changes
Copy link
Member

@KIBORG04 KIBORG04 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Да пойдет, но сомневаюсь только в необходимости полей "дополнительная информация"

@TauKitty TauKitty requested a review from a team September 8, 2024 08:11
@AzzyDreemurr13
Update map_report.yml
575b087 
Принял сторону киборга по вопросу дополнительно
@AzzyDreemurr13
Update map_report.yml
38caca1 
убрал
@TauKitty TauKitty requested a review from a team September 24, 2024 09:53
@AzzyDreemurr13
Copy link
Contributor Author

Капец, за меня обновил, :dovolen:

@AzzyDreemurr13
Update map_report.yml
d1d57a7 
Выбор карт
@AzzyDreemurr13
Update map_report.yml
8c5bb2f 
тест
@AzzyDreemurr13
Update map_report.yml
f8a029c 
Забыл поменять на dropdown
@KIBORG04 KIBORG04 merged commit e74abcc into TauCetiStation:master Sep 29, 2024
12 checks passed
@AzzyDreemurr13 AzzyDreemurr13 deleted the new-template-issue branch September 30, 2024 06:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@KIBORG04 KIBORG04 KIBORG04 approved these changes

@KatoMarika KatoMarika KatoMarika approved these changes

@volas volas Awaiting requested review from volas

Assignees
No one assigned
Labels
Maintainability Improvements Merge Ready
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@AzzyDreemurr13 @TauKitty @LudwigVonChesterfield @volas @KatoMarika @KIBORG04