Skip to content

[logs] fix openapi tags#45

Merged
capcom6 merged 1 commit intomasterfrom
logs/fix-openapi-tags
Apr 10, 2026
Merged

[logs] fix openapi tags#45
capcom6 merged 1 commit intomasterfrom
logs/fix-openapi-tags

Conversation

@capcom6
Copy link
Copy Markdown
Member

@capcom6 capcom6 commented Apr 9, 2026 •
edited by coderabbitai Bot
Loading

Summary by CodeRabbit

  • Chores
    • Improved API schema documentation for date-time field formatting.

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 9, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 9, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 9f725437-0897-4eb4-97d3-f9e97c42fb97

📥 Commits

Reviewing files that changed from the base of the PR and between f83a97a and 45d4d91.

📒 Files selected for processing (1)
  • smsgateway/domain_logs.go

Walkthrough

Updated the LogEntry.CreatedAt struct field tag in the domain logs file by adding format:"date-time" metadata to the JSON serialization tag. This change affects only the field's serialization attributes without modifying any logic or control flow.

Changes

Cohort / File(s) Summary
Struct Field Tag Update
smsgateway/domain_logs.go		 Added format:"date-time" to the LogEntry.CreatedAt field tag alongside the existing json:"createdAt" tag for OpenAPI/JSON Schema compliance.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title '[logs] fix openapi tags' directly relates to the main change: adding the 'format:"date-time"' OpenAPI tag to the LogEntry.CreatedAt field in domain_logs.go.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@capcom6 capcom6 merged commit 6fc2205 into master Apr 10, 2026
8 checks passed
@capcom6 capcom6 deleted the logs/fix-openapi-tags branch April 10, 2026 02:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@capcom6