Closed
Conversation
- 新增本地开发环境配置，包括数据库、Redis和S3设置 - 优化Next.js开发服务器配置，支持0.0.0.0访问 - 改进侧边栏交互逻辑，修复折叠状态下的peek行为 - 添加模型注释和类型定义 - 移除GitHub星标按钮 - 增加并发构建数量
- 新增项目页面列表的布局组件和页面组件 - 在侧边栏导航中添加工作项入口 - 实现页面创建功能及相关权限控制 - 更新.gitignore文件排除开发环境文件 - 添加调试日志和注释说明
将页面列表相关组件重命名为概览列表组件，并更新相应的布局结构 移除旧的页面列表视图，新增概览列表视图组件
feat: 新增项目描述输入组件 feat: 添加项目活动记录组件 feat: 实现项目属性展示组件 feat: 新增工作项统计组件 refactor: 更新项目导航侧边栏 fix: 修复日期组件类型定义 chore: 更新next.js配置 docs: 添加中英文翻译字段
- 在项目功能枚举中添加OVERVIEW选项 - 新增概览页面导航项和侧边栏控制 - 实现概览页面的布局和交互功能 - 更新项目类型定义和主题存储以支持概览功能
Contributor
|
Caution Review failedThe pull request is closed. WalkthroughAdds a Project Overview feature across web UI (new components, navigation, store/service methods, i18n/types), enhances sidebar resize/peek control, introduces project analytics/history fetching, and tweaks UI headers. DevOps updates include Dockerfile for Django API, middleware docker-compose, Next.js config changes, and binding dev servers to 0.0.0.0. Backend settings modified and analytics endpoints extended. Changes
Sequence Diagram(s)sequenceDiagram
autonumber
participant User
participant NextPage as Overview Page
participant Store as ProjectStore
participant Service as ProjectService
participant API as API (Django)
participant UI as Components
User->>NextPage: Navigate /{workspace}/projects/{id}/overview
NextPage->>Store: fetchProjectAnalyze(ws, id)
Store->>Service: getProjectAnalyze(ws, id)
Service->>API: GET /advance-analytics/
API-->>Service: analytics JSON
Service-->>Store: data
Store-->>UI: WorkItemStats props
NextPage->>Store: fetchProjectHistory(ws, id)
Store-->>UI: history data (mock or API)
NextPage->>UI: Render OverviewListView<br/>(Description, Properties, Activity, Stats)
UI-->>User: Overview screen
sequenceDiagram
autonumber
participant User
participant Sidebar as ResizableSidebar
participant Theme as ThemeStore
User->>Sidebar: Hover collapsed rail
Sidebar->>Sidebar: handleTriggerEnter -> show peek
Sidebar->>Theme: overviewSidebarPeek(true)
User->>Sidebar: Move mouse away
Sidebar->>Sidebar: handleTriggerLeave -> hide peek (with timeout)
Sidebar->>Theme: overviewSidebarPeek(false)
User->>Sidebar: Drag resize handle
Sidebar->>Sidebar: startResizing / handleResize (delta, clamp)
Sidebar-->>User: Updated width
User->>Sidebar: Click collapse toggle
Sidebar->>Sidebar: toggleCollapsed -> onCollapsedChange
Estimated code review effort🎯 4 (Complex) | ⏱️ ~60–90 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
✨ Finishing touches
🧪 Generate unit tests
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (41)
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. Comment |
|
yuzheng3 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account. You have signed the CLA already but the status is still pending? Let us recheck it. |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
This PR is being reviewed by Cursor Bugbot
Details
Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.
To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.
| "PORT": 5432, | ||
| } | ||
|
|
||
| } |
There was a problem hiding this comment.
Bug: Security Risk: Hardcoded Sensitive Configurations
Sensitive configuration for the database, Redis, and AWS is hardcoded in the settings file, replacing environment variable usage. This includes database credentials (user, password, host IP), Redis URL, and AWS access keys, creating a security vulnerability and preventing environment-specific deployments.
|
|
||
| # Redis Config | ||
| REDIS_URL = os.environ.get("REDIS_URL") | ||
| REDIS_URL = 'redis://10.32.190.226:6379/0' |
There was a problem hiding this comment.
Bug: Hardcoded Redis URL Exposes Infrastructure
The Redis URL redis://10.32.190.226:6379/0 is hardcoded directly in the settings. This exposes infrastructure details and makes the configuration non-portable, rather than using the REDIS_URL environment variable.
pablohashescobar
added a commit
that referenced
this pull request
Oct 3, 2025
…7887) - Added a new function `save_webhook_log` to log webhook requests and responses to MongoDB, with a fallback to the database if the MongoDB save fails. - Updated the `webhook_send_task` to utilize the new logging function. - Modified the `get_webhook_logs_queryset` to order logs by creation date and adjusted the chunk size for iteration.
sriramveeraghanta
pushed a commit
that referenced
this pull request
Oct 15, 2025
…o database (#7896) * feat: implement webhook logging to MongoDB and fallback to database (#7887) - Added a new function `save_webhook_log` to log webhook requests and responses to MongoDB, with a fallback to the database if the MongoDB save fails. - Updated the `webhook_send_task` to utilize the new logging function. - Modified the `get_webhook_logs_queryset` to order logs by creation date and adjusted the chunk size for iteration. * refactor: clean up log data formatting in save_webhook_log function * fix: update retry_count type in save_webhook_log function
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Type of Change
Screenshots and Media (if applicable)
Test Scenarios
References
Note
Introduce a new project Overview UI (properties, description, activity, work-item stats), extend analytics and services, and add Docker/dev configs for local setup.
.../projects/[projectId]/overview/(list)withOverviewListView, header, and layout; includesProjectDescriptionInput,ProjectProperties,ProjectActivity, andWorkItemStats.overviewentry to project sidebar/breadcrumbs (EProjectFeatureKey.OVERVIEW) and translations (EN/zh-CN).getProjectHistoryandgetProjectAnalyzeAPIs; store actionsfetchProjectHistory(mocked) andfetchProjectAnalyze.undefined, minor UI/image adjustments, comments.overview_view,description_html, relaxed date types).cancelled_work_itemsto work-item stats; minor method signature formatting.apps/api/Dockerfile.django; introduce hardcoded local DB/Redis/MinIO creds, dotenv loading;local.pydefaultSECRET_KEY.docker-compose-middleware.yml(Postgres, Redis, RabbitMQ, MinIO)..gitignoreadditions for venv and misc files.Written by Cursor Bugbot for commit 00a9dbb. This will update automatically on new commits. Configure here.
Summary by CodeRabbit
New Features
Style
Documentation
Chores