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

Удобный ефтпос. #10884

Conversation

DarthSidiousPalpatine
Copy link
Contributor

@DarthSidiousPalpatine DarthSidiousPalpatine commented Feb 27, 2023

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

image
Добавлен девайс, который прикручивается к столу, которым легко пользоваться как продавцу так и пользователю.

Для настройки требуется:

  1. Нажимаем правую нижнюю кнопку.
  2. Вводим номер аккаунта или проводим картой.
  3. Нажимаем зелёную кнопку с кружочком если ввели номер.

upd.
image

Как пользоваться продавцу:

  1. Открываешь интерфейс когда девайс лежит на столе и прикручен.
  2. Нажимаем циферки.
  3. Нажимаем зелёную кнопку с кружочком.
    (ОПЦИОНАЛЬНО) Если хочешь чтобы цена не сбросилась после покупки, а осталась такой же, чтобы тебе не вводить снова эту же цену (допустим. донаты собираем, ввели 5$ и любой желающий просто подходит и может сколько угодно раз заплатить тебе 5$), - нажми на кнопку "Сбрасывать?", чтобы она перестала быть зелёной.

Как пользоваться покупателю:

  1. Увидеть высветившуюся голограмму цены.
  2. Тыкнуть картой по девайсу.

Учтите что интерфейс установки цены откроется только если вы находитесь со стороны клавиатуры. Чтобы покрутить терминал перед тем как положить на стол нажмите на него держа его в руке.

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

Удобнее чем ефтпос.

Авторство

AndreyGysev и все все все, кто помогал в дискорде.

Чеинжлог

🆑

  • rscadd: Добавлен терминал оплаты который придёт на замену EFTPOS.

@TauKitty
Copy link
Contributor

Changelog status: ✔️

@TauKitty TauKitty added Feature Новая фича Map Edit Sprites UI User Interface labels Feb 27, 2023
@TheLone1y
Copy link

Юхууу емать Гисев рвееет!!!

@DarthSidiousPalpatine
Copy link
Contributor Author

Ебать у меня сломался русский язык.

@TauKitty
Copy link
Contributor

Этот запрос на слияние был упомянут в Tau Ceti Station Forum. Там могут быть соответствующие подробности:

https://forum.taucetistation.org/t/fludilka-chast-2/28659/20448

@Pivo24
Copy link
Contributor

Pivo24 commented Feb 27, 2023

А если хочу подключить к EFTPOS карго-аккаунт?

@Chip11-n
Copy link
Contributor

Chip11-n commented Feb 27, 2023

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

Дело вкусов, конечно, однако я думаю, что "сбрасывать?" должно потерять вопрос, или стать "сбросить", или ещё лучше "сброс".

И ещё, не выйдет ли наложения этого большого ценника при двух терминалах рядом?

@DarthSidiousPalpatine
Copy link
Contributor Author

А если хочу подключить к EFTPOS карго-аккаунт?

бля....

@DarthSidiousPalpatine
Copy link
Contributor Author

выйдет и не будет видно. Так что не ставь два терминала рядом :dovolen:

@DarthSidiousPalpatine
Copy link
Contributor Author

так, немного надо переделать кое-что, резонные требования. Не мержить.

@DarthSidiousPalpatine DarthSidiousPalpatine marked this pull request as draft February 27, 2023 14:02
@DarthSidiousPalpatine DarthSidiousPalpatine marked this pull request as ready for review February 27, 2023 15:57
@DarthSidiousPalpatine
Copy link
Contributor Author

Что я сделал. Что-то закодил, оно работает. Даааа....
Собственно, да.

@liteyzz
Copy link

liteyzz commented Feb 27, 2023

крутой ПР, меня только голограмма цены смутила как то выбивается сильно из общего вида

@DarthSidiousPalpatine
Copy link
Contributor Author

Ну кароч готово

Copy link
Contributor

@Sakuya-Izayoi Sakuya-Izayoi left a comment

Choose a reason for hiding this comment

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

нельзя ввести комментарий к переводу как в оригинальном ефтпосе, даунгрейд

@LudwigVonChesterfield
Copy link
Contributor

35 Runtime in cardpay.dm:132 : Cannot read null.security_level

@DarthSidiousPalpatine
Copy link
Contributor Author

  • когда вводишь номер счёта в интерфейсе пишет "0" до того как ты что-то введёшь, немного путает.
  • обычно у нас номер счёта начинается с # а не №

Насколько сложно эти две вещи пофиксить?

Кароч вот это я поправил нолик. # знак я не хочу, имхо.

Ну и баг оказался сложнее, я не понял от чего он вообще и как. хз.

@LudwigVonChesterfield
Copy link
Contributor

Баг это вот этот рантайм? 35 Runtime in cardpay.dm:132 : Cannot read null.security_level

@LudwigVonChesterfield
Copy link
Contributor

Если поменять № на # (потому-что номера счёта везде в игре именно так обозначаются, а не потому-что я хочу подавить твою креативность, я не против если ты потом сделаешь отдельный ПР где все упоминания номера счёта поменяешь на этот символ)

И если пофиксить рантайм (написал под конкретной строкой в чем думаю проблема)

то думаю что ПР готов к мержу.

@LudwigVonChesterfield
Copy link
Contributor

обнаружили прикол что на терминал оплаты нет КД на клики и можно спаммить звуками к примеру если кликать карточкой по не настроенному терминалу

@DarthSidiousPalpatine
Copy link
Contributor Author

Ну итак, я заменил #, я добавил проверку get_account(linked_account), сделал кд на клики, сделал ранние возвраты.

@DarthSidiousPalpatine
Copy link
Contributor Author

Думаю, это всё, пойду форкнусь от этой ветки и сделаю интерфейс в другом пре...

@LudwigVonChesterfield
Copy link
Contributor

LudwigVonChesterfield commented Jun 3, 2023

как только тут решат конфликт, поправят(или уберут) спрайт голограммы и удалят старый ЕФТПОС - смержу ПР.

@DarthSidiousPalpatine
Copy link
Contributor Author

как только тут решат конфликт, поправят(или уберут) спрайт голограммы и удалят старый ЕФТПОС - смержу ПР.

ГоТоВо

@DarthSidiousPalpatine
Copy link
Contributor Author

GIF 03 06 2023 11-41-40

@DarthSidiousPalpatine
Copy link
Contributor Author

image
image
image

Переделал на три штуки крч.

@LudwigVonChesterfield LudwigVonChesterfield merged commit 378f2d9 into TauCetiStation:master Jun 4, 2023
10 checks passed
TauKitty added a commit to TauCetiStation/ClassicChangelog that referenced this pull request Jun 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Новая фича Map Edit Sprites Test Merge Candidate ПР с этим лэйблом будет или уже находится с тест мерже UI User Interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

10 participants