This PR adds both Chinese and English versions of the README.md file to improve the project's accessibility and global reach. #1049
Conversation
This file contains 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
- 修改多个配置文件以更新数据库连接字符串 - 添加本地时区设置 - 调整服务配置以支持 PostgreSQL - 移除各个模块独立的数据库配置，使其支持当项目启动数据迁移
colinin
reviewed
Dec 11, 2024
Comment on lines
+45
to
+51
| public class YourNotificationHandler : INotificationHandler<SessionExpirationNotification> | ||
| { | ||
| public async Task HandleNotificationAsync(SessionExpirationNotification notification) | ||
| { | ||
| // 处理会话过期通知 | ||
| } | ||
| } |
There was a problem hiding this comment.
通知模块没有集成 MediatR, 无法使用此API, 不能在文档中诱导用户
colinin
reviewed
Dec 11, 2024
Comment on lines
+32
to
+36
| "Claims": { | ||
| "DynamicClaims": { | ||
| "OrganizationUnit": true // 启用组织单元动态声明 | ||
| } | ||
| } |
colinin
reviewed
Dec 11, 2024
Comment on lines
+39
to
+40
| "WrapOnError": true, // 是否包装错误响应 | ||
| "WrapOnSuccess": true // 是否包装成功响应 |
colinin
reviewed
Dec 11, 2024
| * Actor方法的错误响应会保持原始错误结构 | ||
| * 模块与ABP的远程服务错误处理系统集成 | ||
| * 建议在微服务架构中统一使用响应包装 | ||
| * 包装格式可以通过继承`IAbpWrapperResponseBuilder`自定义 |
There was a problem hiding this comment.
使用IActionResultWrapperFactory和IActionResultWrapper
colinin
reviewed
Dec 11, 2024
Comment on lines
+36
to
+37
| "WrapOnError": true, // 是否包装错误响应 | ||
| "WrapOnSuccess": true // 是否包装成功响应 |
colinin
reviewed
Dec 11, 2024
| * `Field` - 字段名 | ||
| * `Value` - 字段值 | ||
| * `Operate` - 操作符 | ||
| * `IsLeft` - 是否左括号 |
There was a problem hiding this comment.
目前项目模块太多了，如果对于每一个项目中的readme进行工人校验成本太大。我打算通过ai把所有模块生成readme后再去docs文件夹汇总各个模块组的readme。至于通过AI生成的项目级别的readme可以再后期逐步维护
|
也可以，没有核对的README可以备注一下内容来自模型，如有差异请提issues或PR
Colin
***@***.***
…------------------ 原始邮件 ------------------
发件人: ***@***.***>;
发送时间: 2024年12月11日(星期三) 下午2:59
收件人: ***@***.***>;
抄送: ***@***.***>; ***@***.***>;
主题: Re: [colinin/abp-next-admin] This PR adds both Chinese and English versions of the README.md file to improve the project's accessibility and global reach. (PR #1049)
@feijie999 commented on this pull request.
In aspnet-core/framework/data-protection/LINGYUN.Abp.DataProtection.Abstractions/README.md:
> +## 数据操作类型 + +* `DataAccessOperation.Read` - 查询操作 +* `DataAccessOperation.Write` - 更新操作 +* `DataAccessOperation.Delete` - 删除操作 + +## 数据过滤 + +* `DataAccessFilterLogic` - 数据过滤逻辑 + * `And` - 且 + * `Or` - 或 +* `DataAccessFilterRule` - 数据过滤规则 + * `Field` - 字段名 + * `Value` - 字段值 + * `Operate` - 操作符 + * `IsLeft` - 是否左括号
目前项目模块太多了，如果对于每一个项目中的readme进行工人校验成本太大。我打算通过ai把所有模块生成readme后再去docs文件夹汇总各个模块组的readme。至于通过AI生成的项目级别的readme可以再后期逐步维护
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: ***@***.***>
|
|
已完成对所有的模块项目添加中英文readme.md。下一步计划考虑给每一个项目readme.md加上由AI生成的提示，这我需要写一个工具来读取整个仓库的所有readme.md，然后再恰当的位置写入提示”注意:此内容为AI模型分析代码后自动生成,如有差异请提issues或PR“ |
(cherry picked from commit 239ad14)
🔄 rename(appsettings.Development.json): 重命名开发环境配置文件为 appsettings.MySql.json 🔧 update(SingleMigrationsDbContextFactory.cs): 更新数据库上下文工厂以使用 MySql 配置文件
|
此分支还有要提交的吗,没有的话我这两天看看合并了 |
这个分支主要是readme的编写，可以只合并readme相关的提交。另外一个main分支的pr是用于在单项目启动时支持pgsql和sqlserver的支持，包括一些启动说明的readme |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Key points: