Refactor: rename /chat_channels to /chat-channels by wangq8 · Pull Request #16099 · infiniflow/ragflow (original) (raw)

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info ⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 523e0641-3506-4977-9a2a-9c50b99cd986

📥 Commits

Reviewing files that changed from the base of the PR and between 15f50e5 and 0a71199.

📒 Files selected for processing (2)


📝 Walkthrough

Walkthrough

All chat channel REST API route paths are renamed from underscore-separated (/chat_channels) to hyphen-separated (/chat-channels). The change is applied to four Flask route decorators in the backend and five URL string constants in the frontend TypeScript API mapping. No handler logic or function signatures are modified.

Changes

Chat Channel Route Path Rename

Layer / File(s) Summary
Rename /chat_channels to /chat-channels across backend and frontend api/apps/restful_apis/chat_channel_api.py, web/src/utils/api.ts The @manager.route decorators for POST, GET, PATCH, and DELETE in chat_channel_api.py, and the chatChannelSet, chatChannelList, chatChannelDetail, chatChannelUpdate, and chatChannelDel URL strings in api.ts, are all updated from /chat_channels to /chat-channels.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐇 A dash replaced the underline,
in paths both front and back,
/chat-channels now aligned,
keeping everything on track.
Hop hop, the rabbit cheers — hyphen wins the race! 🎉


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.