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

add: health insurance #5921

Merged
merged 22 commits into from
Oct 26, 2024
Merged

add: health insurance #5921

merged 22 commits into from
Oct 26, 2024

Conversation

Anorak2024
Copy link
Contributor

@Anorak2024 Anorak2024 commented Sep 18, 2024 •
edited
Loading

Описание

Мед страховка. У разных ролей может быть разная страховка. В зависимости от страховки раундстартом даются поинты страховки. При анализе здоровья можно списать из страховки стоимость лечения в зависимости от повреждений. Недостающие поинты снимаются в виде кредитов. Можно пополнять страховку и менять ее тип за доплату. В отличии от ТЗ нет никаких доп консолей, все перенесено в банкоматы.

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

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

Тесты

  1. Тыкнул сканером здоровья, списал страховку с избитой куклы.
  2. Списал страховку с частичными затратами обычных кредитов
  3. Пополнил страховку в автомате.
  4. Сменил тип страховки
  5. Попробовал списать страховку когда ее не хватает.

@github-actions github-actions bot added TGUI Affects TGUI javascript Pull requests that update Javascript code labels Sep 18, 2024
@Anorak2024 Anorak2024 marked this pull request as ready for review September 19, 2024 19:54
@ROdenFL
Copy link
Contributor

ROdenFL commented Sep 20, 2024

Какая нахуй страховка. Вы че там ебанулись?)

@Anorak2024
Copy link
Contributor Author

Какая нахуй страховка. Вы че там ебанулись?)

Немножко. Это вроде еще при тебе прошло.

@BoshyG
Copy link
Contributor

BoshyG commented Sep 21, 2024

Поправка для тех кто будет мерджить, нужно тестом вместе с зарплатами заливать, не в основную ветку. Мы будем вводить постепенно зарплаты нам бы хорошо чтобы страховка так же шла паралельно зарплатам

@Anorak2024
Copy link
Contributor Author

@Bizzonium, может это вместе с зарплатами в тестмерж? Связанные предложки, экономика апдейт, все такое.

@Bizzonium Bizzonium added the testmerge This PR is being tested right now on live test servers. label Sep 29, 2024
@Bizzonium
Copy link
Member

Конфликт есть в этом файле
image

@Anorak2024
Copy link
Contributor Author

Конфликт есть в этом файле image

Убрал аргумент который не исспользовал, а так-же вызов функции с его передачей. Щас не должно друг друга ломать.

@Bizzonium
Copy link
Member

Конфликтов много всё равно. Плохая идея вот так совмещать. Придётся подождать

@Bizzonium Bizzonium removed the testmerge This PR is being tested right now on live test servers. label Sep 29, 2024
@BoshyG
Copy link
Contributor

BoshyG commented Sep 29, 2024 •
edited
Loading

@NightDawnFox Зюзя в чем тут траблы? Где ботаник накакал?

@Anorak2024
Copy link
Contributor Author

@NightDawnFox Зюзя в чем тут траблы? Где ботаник накакал?

Траблы в том что у нас мерж конфликты которые ни я ни пельмень не можем нормально пофиксить пока один из ПРов не замержат, так-что пока в тестмерже ПР пельменя по праву первого пинга.

@github-actions github-actions bot added the Merge Conflict This doesn't seem right label Oct 5, 2024
@Bizzonium Bizzonium added the testmerge This PR is being tested right now on live test servers. label Oct 5, 2024
github-actions bot pushed a commit that referenced this pull request Oct 6, 2024
@actions-user
add: health insurance (#5921) [testmerge][94f4538]
c28adca
github-actions bot pushed a commit that referenced this pull request Oct 6, 2024
@actions-user
add: health insurance (#5921) [testmerge][94f4538]
0749e41
github-actions bot pushed a commit that referenced this pull request Oct 6, 2024
@actions-user
add: health insurance (#5921) [testmerge][94f4538]
18867ac
github-actions bot pushed a commit that referenced this pull request Oct 6, 2024
@actions-user
add: health insurance (#5921) [testmerge][94f4538]
29ea60a
github-actions bot pushed a commit that referenced this pull request Oct 6, 2024
@actions-user
add: health insurance (#5921) [testmerge][94f4538]
5e7e138
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Oct 6, 2024

Данный PR был добавлен в ветку "testmerge" для тестов. Он попадёт на сервер после деплоя.

github-actions bot pushed a commit that referenced this pull request Oct 6, 2024
@actions-user
add: health insurance (#5921) [testmerge][94f4538]
992415b
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Oct 7, 2024

Данный PR был добавлен в ветку "testmerge" для тестов. Он попадёт на сервер после деплоя.

github-actions bot pushed a commit that referenced this pull request Oct 7, 2024
@actions-user
add: health insurance (#5921) [testmerge][94f4538]
62f7822
Bizzonium pushed a commit that referenced this pull request Oct 7, 2024
@actions-user @Bizzonium
add: health insurance (#5921) [testmerge][94f4538]
184ec94
@Anorak2024
Copy link
Contributor Author

@Bizzonium, багу пофиксил, надо бы залить, а то можно бесконечную страховку фармить из-за того что кровь может немного превышать лимит.

github-actions bot pushed a commit that referenced this pull request Oct 9, 2024
@actions-user
add: health insurance (#5921) [testmerge][cdf3898]
a2a742a
github-actions bot pushed a commit that referenced this pull request Oct 9, 2024
@actions-user
add: health insurance (#5921) [testmerge][cdf3898]
d7ea518
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Oct 9, 2024

Данный PR был добавлен в ветку "testmerge" для тестов. Он попадёт на сервер после деплоя.

@github-actions GitHub Actions
Copy link
Contributor

Данный PR был добавлен в ветку "testmerge" для тестов. Он попадёт на сервер после деплоя.

github-actions bot pushed a commit that referenced this pull request Oct 13, 2024
@actions-user
add: health insurance (#5921) [testmerge][7aeddd7]
5846cf8
github-actions bot pushed a commit that referenced this pull request Oct 18, 2024
@actions-user
add: health insurance (#5921) [testmerge][a4236a6]
af65167
@github-actions GitHub Actions
Copy link
Contributor

Данный PR был добавлен в ветку "testmerge" для тестов. Он попадёт на сервер после деплоя.

github-actions bot pushed a commit that referenced this pull request Oct 23, 2024
@actions-user
add: health insurance (#5921) [testmerge][a4236a6]
54ae86b
@github-actions GitHub Actions
Copy link
Contributor

Данный PR был добавлен в ветку "testmerge" для тестов. Он попадёт на сервер после деплоя.

github-actions bot pushed a commit that referenced this pull request Oct 23, 2024
@actions-user
add: health insurance (#5921) [testmerge][a4236a6]
d8bd880
@github-actions GitHub Actions
Copy link
Contributor

Данный PR был добавлен в ветку "testmerge" для тестов. Он попадёт на сервер после деплоя.

@Bizzonium Bizzonium merged commit 29f373e into ss220-space:master220 Oct 26, 2024
12 checks passed
@Bizzonium Bizzonium removed the testmerge This PR is being tested right now on live test servers. label Oct 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Den109G Den109G Den109G left review comments

Assignees
No one assigned
Labels
javascript Pull requests that update Javascript code TGUI Affects TGUI
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@Anorak2024 @ROdenFL @BoshyG @Bizzonium @Den109G