Merge pull STCS #2
No Assignees
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: kodorvan/stcs#2
Reference in New Issue
Block a user
No description provided.
Delete Branch "Pixe1lz/stcs:stable"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Где теги?
Это что такое?
Ой, это я для себя делал, чтобы в консоли видеть json файлы и данные, чтобы они были отдельны друг от друга вот так их отделял
Ничего сделанного для себя не должно попадать в историю кода, в общий репозиторий
Кнопка "Торговать" заменить на "Начать торговлю 🔥" или что-либо похожее
Раздел "о себе" удалить
После того как я нажал на кнопку "Настройки" показало сообщение: "Регистрация прошла успешно, здравствуйте mirzaev_sexy!" - такого быть не должно, ведь я зарегистрировался уже давно"
Видимо в базе данных что-то не записалось в тот раз, во второй раз показывает уже: "С возвращением, mirzaev_sexy!"
Домены должны начинаться с @, либо используй имя фамилия
Кнопку "Начать торговлю" замени на "Запуск"
При нажатии на кнопку ничего не происходит
Должна быть проверка, что если ни одна платформа не подключена, то будет отправлять сообщение "подключите платформу"
У меня добавлено отладочное сообщение "Баланс не был получен", который говорит о том, что не было подключено платформы
Кнопка настройки должна быть снизу, а запуск сверху и выделяться. Важные кнопки всегда вверху.
Во-первых это абсолютно непонятное сообщение для пользователя
Во-вторых этого сообщения у меня не приходит
Исправлено:
Использовать для приветствия имя пользователя, а не его домен. (исправлено)
Не бибут, а байбит (исправлено)
Не "бот по трейдингу", а "чат-робот для трейдинга" (исправлено)
"Здесь вы получите" с новой строки, над списком (исправлено)
"Просто отправляйте интересующий вас инструмент..." весь абзац удалить (удалено)
Кнопка "Торговать" заменить на "Начать торговлю 🔥"
Была заменена на "🔥 Начать торговлю
Раздел "о себе" удалить (удалено)
После того как я нажал на кнопку "Настройки" показало сообщение: "Регистрация прошла успешно, здравствуйте mirzaev_sexy!" - такого быть не должно, ведь я зарегистрировался уже давно"
Был заменен текст на:
"Регистрация прошла успешно, перейдите в профиль нажав на кнопку!'"
Она не отправлялась второй раз, просто основное сообщение где указан баланс, настройки не отправляются новым сообщением, они редактируются, а сообщение об регистрации просто сверху висит.
Была заменена кнопка на "Запуск", также изменено сообщение об ошибке на "Баланс не был получен, подключите платформу"
P.s. Код был почищен от текстов и комментариев сделанных для себя.
Удален лишний файл app/services/bybit/config.py.
Была очищена база данных для новой проверки.
8 пункт
Сообщение о том что я зарегистрировался появилось после нажатия на кнопку "Настройки", тебе об этом было сказано напрямую. Оно не появилось когда я зарегистрировался, оно не "висит сверху перед другими сообщениями". Оно было отправлено посреди уже моего пользования роботом, его там быть не должно.
Текст менять я не просил. Тем более никакой регистрации у нас нет. Какая вообще регистрация?
НИКОГДА НЕ ЗАГРУЖАЙ БАЗУ ДАННЫХ В РЕПОЗИТОРИЙ
если ты имел ввиду для тестирования робота, то порядок
У нас не только ByBit, будут другие платформы. Просто чат-робот для автоматизации трейдинга
"Здесь вы получите:" удали
Регистрация прошла успешно, перейдите в профиль нажав на кнопку!
удалить сообщение. никакой регистрации нет. суматоху наводит ненужную
к тому же оно просит перейти в профиль когда ты уже перешёл в профиль
Когда я жму "Запуск", то у меня открывается окно настройки байбит. Этого не должно быть.
"Баланс не был получен" это информация абсолютно бесполезная и сбивающая с толку для пользователя
Должно быть написано: "⚠️ Подключите платформу для торговли"
Да и вообще кнопка запуска должна быть скрыта, если платформы ни одной ещё не подключено
Arsen Mirzaev, [21.07.2025 19:00]
У тебя нет сообщений о том, что настройка изменена
Arsen Mirzaev, [21.07.2025 19:01]
Пользователь не понимает отредактировал настройку или нет
Вот пример как должно выглядеть
Внесены исправления:
1. В текст приветственного сообщения из 'Чат-робот по трейдингу bybit' --> 'чат-робот для автоматизации трейдингу'
2. В текст приветственного сообщения были удалены абзацы после фразы 'Здесь вы получите', фраза тоже удалена
3. Текст 'Регистрация прошла успешно, перейдите в профиль нажав на кнопку!' был заменен на 'Здравствуйте, {username}'
Почему я просто не уберу его? Потому что к тексту 'Здравствуйте, {username}' привязана reply клавиатура ('👤 Профиль'), а к тексту профиля привязана inline клавиатура ('Настройки', 'Запуск'). Две клавиатуры к одному сообщению нельзя привязать, поэтому я сделал такой обход.
4. Был удален текст после повторного нажатия на кнопку "🔥 Начать торговлю" ('С возвращением, {username}')
5. В БД изменил значение по умолчанию для "Максимальный риск на сделку (% от баланса)" с 1 до 100
6. В настройках 'Риск-менеджмент' добавил в конце каждого значения символ '%'
Все проверено и работает без ошибок.
Внесены исправления:
1. При нажатии на кнопку "Запуск", не будет выводиться сообщение с профилем bybit, пока пользователь не подключит его в настройках.
Добавлены сообщения-исключения пользователю, которые вызываются, если:
2. Добавлены сообщения об успешном изменении настроек (скриншот 1)
3. Добавлены сообщения об не успешном изменении настроек (скриншот 2)