fix: make NotificationConfigBase.name optional with default 'My Notification' to match DB default
Agent-Logs-Url: https://github.com/christianlouis/InboxConverge/sessions/7640aaa6-61f4-4c65-8eb7-f33c2d3eb755 Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com>
This commit is contained in:
@@ -14,6 +14,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
|
||||
- `GET /processing-runs` (was: `GET /processing-runs/processing-runs`)
|
||||
- `GET /processing-runs/{id}` (was: `GET /processing-runs/processing-runs/{id}`)
|
||||
- `GET /processing-runs/{id}/logs` (was: `GET /processing-runs/processing-runs/{id}/logs`)
|
||||
- **`NotificationConfigCreate` schema test failure**: `NotificationConfigBase.name` was a required field (`...`) but the unit test and the database column both use a default of `"My Notification"`. Changed the Pydantic field to `default="My Notification"` to match the DB default and allow callers to omit the field.
|
||||
|
||||
### Added
|
||||
- **Semantic Release** (`release.yml`): Automated versioning and GitHub Release creation on every push to `main` using `python-semantic-release`. Reads conventional-commit prefixes (`feat:`, `fix:`, etc.) to determine the next version and updates `CHANGELOG.md`.
|
||||
|
||||
@@ -290,7 +290,9 @@ class PaginatedAdminLogsResponse(BaseModel):
|
||||
# Notification Config Schemas
|
||||
class NotificationConfigBase(BaseModel):
|
||||
name: str = Field(
|
||||
..., max_length=255, description="Friendly name for this notification channel"
|
||||
default="My Notification",
|
||||
max_length=255,
|
||||
description="Friendly name for this notification channel",
|
||||
)
|
||||
channel: NotificationChannel
|
||||
apprise_url: Optional[str] = Field(None, description="Apprise notification URL")
|
||||
|
||||
Reference in New Issue
Block a user