Skip to content

Conversation

@timtian
Copy link
Contributor

@timtian timtian commented Dec 17, 2025

English | 简体中文

PR

PR Checklist

请检查您的 PR 是否满足以下要求:

  • commit message遵循我们的提交贡献指南
  • 添加了更改内容的测试用例(用于bugfix/功能)
  • 文档已添加/更新(用于bugfix/功能)
  • 是否构建了自己的设计器,经过了充分的自验证

PR 类型

这个PR的类型是?

  • 日常 bug 修复
  • 新特性支持
  • 代码风格优化
  • 重构
  • 构建优化
  • 测试用例
  • 文档更新
  • 分支合并
  • 其他改动(请补充)

需求背景和解决方案

当使用mock模式时,
国际化内容的保存修改。并没有持久化,导致刷新后修改内容消失,同时由于在preview时由于拉取的i18n的位置并不一致导致,预览时也无法预览国际化后的内容

持久化没月使用nodb进行持久化,而是直接针对schema/1.json进行保存修改,预览时拉取的 i18n位置也统一至 1.json.确保一致

Issue Number: N/A

修改前

修改后

此PR是否含有 breaking change?

Other information

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 17, 2025

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

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

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.

@github-actions github-actions bot added the enhancement New feature or request label Dec 17, 2025
Comment on lines +142 to +143
created_at: getRandomDate(),
updated_at: getRandomDate()
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这里为什么要随机日期时间呢?可以按照实际的日期时间?

import { getResponseData } from '../tool/Common'

const schemaFilePath = path.resolve(process.cwd(), './src/mock/get/app-center/v1/apps/schema/1.json')
export default class PageService {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这里类名需要重命名一下,比如: I18nService

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants