Skip to content

feat: add Codex channel disclaimer (i18n, OpenAI terms) - #2852

Merged
Calcium-Ion merged 4 commits into
QuantumNous:mainfrom
seefs001:fix/codex-tips
Feb 7, 2026
Merged

feat: add Codex channel disclaimer (i18n, OpenAI terms)#2852
Calcium-Ion merged 4 commits into
QuantumNous:mainfrom
seefs001:fix/codex-tips

Conversation

@seefs001

@seefs001 seefs001 commented Feb 5, 2026

Copy link
Copy Markdown
Collaborator

Summary by CodeRabbit

  • New Features

    • Added an in-modal warning banner for a specific channel configuration
    • Added multilingual disclaimer translations (EN, FR, JA, RU, VI, ZH)
    • Added a new Codex model option to the available model list
  • Improvements

    • Simplified usage logs details column rendering for clearer content display

@coderabbitai

coderabbitai Bot commented Feb 5, 2026

Copy link
Copy Markdown
Contributor

Caution

Review failed

The pull request is closed.

Walkthrough

Adds a conditional warning Banner to the Edit Channel modal for channel type 57 (OAuth JSON), simplifies rendering in the Usage Logs DETAILS column to render computed content directly, adds the same Chinese-disclaimer translation key across six locale files, and appends "gpt-5.3-codex" to a codex model list.

Changes

Cohort / File(s) Summary
Channel Modal UI Updates
web/src/components/table/channels/modals/EditChannelModal.jsx
Added a conditional warning Banner when channel.type === 57 (OAuth JSON). Modified header_override example formatting/indentation only (no functional change).
Usage Logs Display
web/src/components/table/usage-logs/UsageLogsColumnDefs.jsx
Replaced the previous composite summary construction with direct rendering of the computed content in the DETAILS column (removed the prior summary concatenation logic).
Internationalization
web/src/i18n/locales/en.json, web/src/i18n/locales/fr.json, web/src/i18n/locales/ja.json, web/src/i18n/locales/ru.json, web/src/i18n/locales/vi.json, web/src/i18n/locales/zh.json
Added a new Chinese-disclaimer translation key to all six locale files with language-specific translations (new i18n entries only).
Codex Model List
relay/channel/codex/constants.go
Added "gpt-5.3-codex" to baseModelList, thereby including it in the derived model list; no API signature changes.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Poem

🐰 I nibbled lines and left a sign,
A banner pops where OAuth aligns,
Logs now whisper plainly, not a weave,
Six tongues warn users—do believe,
The rabbit hops off, code tidy and fine.

🚥 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 accurately describes the primary change: adding a Codex channel disclaimer with internationalization support and OpenAI terms compliance.
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.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 ast-grep (0.40.5)
web/src/components/table/channels/modals/EditChannelModal.jsx

[]


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.

@Calcium-Ion
Calcium-Ion merged commit 78ddb85 into QuantumNous:main Feb 7, 2026
1 check was pending
ennnnny pushed a commit to ennnnny/new-api that referenced this pull request Mar 17, 2026
feat: add Codex channel disclaimer (i18n, OpenAI terms)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants