feat: Model updates - add deleted_at, timezone, admin_user_ids fields #56

shoko · 2026-04-04T07:00:07+02:00

shoko commented

2026-04-04 07:00:07 +02:00

Summary

Issue #41: Model updates for Phase 2 features

Bounty model changes:

Add deleted_at: int | None - timestamp when deleted (soft-delete support)
Add created_by_username: str | None - username for display purposes

RoomData model changes:

Add timezone: str | None - room timezone (e.g., "Asia/Jakarta")
Add admin_user_ids: list[int] - list of admin user IDs

Changes

core/models.py: Updated Bounty and RoomData dataclasses
adapters/storage/json_file.py: Updated load/save to handle new fields
tests/test_models.py: Added tests for new fields

Testing

All 93 tests pass.

## Summary Issue #41: Model updates for Phase 2 features ### Bounty model changes: - Add `deleted_at: int | None` - timestamp when deleted (soft-delete support) - Add `created_by_username: str | None` - username for display purposes ### RoomData model changes: - Add `timezone: str | None` - room timezone (e.g., "Asia/Jakarta") - Add `admin_user_ids: list[int]` - list of admin user IDs ### Changes - `core/models.py`: Updated Bounty and RoomData dataclasses - `adapters/storage/json_file.py`: Updated load/save to handle new fields - `tests/test_models.py`: Added tests for new fields ### Testing All 93 tests pass.

shoko added 1 commit 2026-04-04 07:00:08 +02:00

feat: add deleted_at, created_by_username to Bounty; timezone, admin_user_ids to RoomData fee8504813

Issue #41: Model updates for Phase 2 features

Bounty model:
- Add deleted_at: int | None - timestamp when deleted (soft-delete)
- Add created_by_username: str | None - username for display purposes

RoomData model:
- Add timezone: str | None - room's timezone (e.g., "Asia/Jakarta")
- Add admin_user_ids: list[int] - list of admin user IDs

Storage adapter updated to handle new fields in load/save operations.
Tests added for new fields.

han approved these changes 2026-04-04 07:07:41 +02:00

han left a comment

lgtm

shoko merged commit d413f6ce13 into main

2026-04-04 07:08:28 +02:00

shoko referenced this issue from a commit

2026-04-04 07:08:30 +02:00

Merge pull request 'feat: Model updates - add deleted_at, timezone, admin_user_ids fields' (#56) from fix/issue-41 into main

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: shoko/jigaido#56