killingdruid/telegram-tui
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

refactor #24

Merged
killingdruid merged 4 commits from refactor into main 2026-03-02 22:00:07 +00:00
Conversation 0 Commits 4 Files Changed 15 +219 -198
killingdruid commented 2026-03-02 21:58:24 +00:00
Owner
Copy Link

Описание

Краткое описание изменений в этом PR.

Тип изменений

  • Bug fix (исправление бага)
  • New feature (новая функция)
  • Breaking change (изменение, ломающее обратную совместимость)
  • Refactoring (рефакторинг без изменения функциональности)
  • Documentation (изменения в документации)
  • Performance improvement (улучшение производительности)

Связанные Issue

Fixes #(номер issue)

Как протестировано?

Опишите тесты, которые вы провели:

  • Тест A
  • Тест B
  • Тест C

Сценарии тестирования

Подробные шаги для проверки изменений:

  1. Запустить cargo run
  2. Сделать X
  3. Убедиться, что Y

Чеклист

  • Мой код следует стилю проекта
  • Я запустил cargo fmt
  • Я запустил cargo clippy и исправил warnings
  • Код компилируется без ошибок (cargo build)
  • Я протестировал изменения вручную
  • Я обновил документацию (если необходимо)
  • Я добавил тесты (если применимо)
  • Все существующие тесты проходят

Скриншоты (если применимо)

Добавьте скриншоты для демонстрации UI изменений.

Дополнительные заметки

Любая дополнительная информация для ревьюверов.

## Описание Краткое описание изменений в этом PR. ## Тип изменений - [ ] Bug fix (исправление бага) - [ ] New feature (новая функция) - [ ] Breaking change (изменение, ломающее обратную совместимость) - [ ] Refactoring (рефакторинг без изменения функциональности) - [ ] Documentation (изменения в документации) - [ ] Performance improvement (улучшение производительности) ## Связанные Issue Fixes #(номер issue) ## Как протестировано? Опишите тесты, которые вы провели: - [ ] Тест A - [ ] Тест B - [ ] Тест C ## Сценарии тестирования Подробные шаги для проверки изменений: 1. Запустить `cargo run` 2. Сделать X 3. Убедиться, что Y ## Чеклист - [ ] Мой код следует стилю проекта - [ ] Я запустил `cargo fmt` - [ ] Я запустил `cargo clippy` и исправил warnings - [ ] Код компилируется без ошибок (`cargo build`) - [ ] Я протестировал изменения вручную - [ ] Я обновил документацию (если необходимо) - [ ] Я добавил тесты (если применимо) - [ ] Все существующие тесты проходят ## Скриншоты (если применимо) Добавьте скриншоты для демонстрации UI изменений. ## Дополнительные заметки Любая дополнительная информация для ревьюверов.
killingdruid added 4 commits 2026-03-02 21:58:24 +00:00
refactor: consolidate message loading logic into chat_loader.rs ec2758ce18 
Move all three phases of chat message loading from scattered locations
into a single dedicated module for better cohesion and navigability:
- Phase 1: open_chat_and_load_data (from handlers/chat_list.rs)
- Phase 2: process_pending_chat_init (extracted from 70-line inline block in main.rs)
- Phase 3: load_older_messages_if_needed (from handlers/chat.rs)

No behaviour changes — pure refactoring.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
test: update footer snapshots to always show account name df89c4e376 
Snapshots now reflect the new behaviour where the account indicator
is always visible (including "default"), matching the footer.rs change.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
style: replace DarkGray with Rgb(160,160,160) for better terminal compatibility 260b81443e 
DarkGray renders differently across terminals; a specific RGB value gives
consistent appearance. Also always show the account indicator in the footer.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
docs: remove out-of-scope items from Phase 14 Etap 4 roadmap
Some checks failed
ci/woodpecker/pr/check Pipeline failed
Details
dea3559da7 
Remove account deletion from modal and parallel polling — these won't
be implemented in the current scope.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
killingdruid merged commit de18d6978b into main 2026-03-02 22:00:07 +00:00
killingdruid deleted branch refactor 2026-03-02 22:00:07 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
killingdruid
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: killingdruid/telegram-tui#24
Delete Branch "refactor"

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?