Niutrans support #226
Niutrans support #226
Conversation
| @@ -167,6 +167,6 @@ SPEC CHECKSUMS: | |||
| Sparkle: 270cd27377bf04e9c128af06e3a22d0f572d6ee3 | |||
| SSZipArchive: 62d4947b08730e4cda640473b0066d209ff033c9 | |||
|
|
|||
| PODFILE CHECKSUM: 851af542ac1369ffd80ab7b91dc95f158ee0d3d6 | |||
| PODFILE CHECKSUM: 3e98cd87eb9495431dc2cd105c1571edc8f60360 | |||
| @@ -223,7 +223,7 @@ - (instancetype)init { | |||
| return nil; | |||
| } | |||
|
|
|||
| self.acceptableContentTypes = [NSSet setWithObjects:@"application/json", @"text/json", @"text/javascript", nil]; | |||
| self.acceptableContentTypes = [NSSet setWithObjects:@"application/json", @"text/json", @"text/javascript", @"text/javascript", @"text/html", nil]; | |||
There was a problem hiding this comment.
一般情况，不应该手动修改第三方 Pod 库的，请问你这里修改是为什么？
There was a problem hiding this comment.
这个想让AFNetworking 能够接收text/html格式返回数据。
There was a problem hiding this comment.
不不，不需要改源码，acceptableContentTypes 有对外的接口的，参考这里
There was a problem hiding this comment.
之前我看了这个接口请求，是 JOSN 格式，也没有用到 text/html
| @@ -1969,7 +1981,7 @@ | |||
| }; | |||
| buildConfigurationList = C99EEB132385796700FEE666 /* Build configuration list for PBXProject "Easydict" */; | |||
| compatibilityVersion = "Xcode 13.0"; | |||
| developmentRegion = en; | |||
| developmentRegion = "zh-Hans"; | |||
There was a problem hiding this comment.
请问这个是在哪里修改的，为什么修改？
Easydict.xcodeproj/project.pbxproj 配置文件不一般不要随便修改。
| @@ -1,16 +1,9 @@ | |||
| { | |||
| "sourceLanguage" : "en", | |||
| "sourceLanguage" : "zh-Hans", | |||
There was a problem hiding this comment.
Easydict/App/mul.lproj/Main.xcstrings 这个文件也是，我记得添加一个新服务，好像是不需要修改这个文件的。
| @@ -25,7 +25,7 @@ | |||
| ); | |||
| dependencies = ( | |||
| 20000DB20840BFBA5AE529DA48FF7827 /* PBXTargetDependency */, | |||
| 0E9A554FD59787BBD43C562B29547AFF /* PBXTargetDependency */, | |||
| 457A6020E71E493D7145A8EC8874C5BB /* PBXTargetDependency */, | |||
There was a problem hiding this comment.
这个文件也不应该修改，你是执行了 pod update，或是接受了系统的建议更改提示？
There was a problem hiding this comment.
确实在过程中执行了pod update
There was a problem hiding this comment.
@BigGuang97 不需要执行 pod update，你这个 PR 改了太多无关的东西，你重新整理一下。
你这个分支，编译就报错，Niutrans 目录下的文件没有关联到 target
参考这里的编译步骤，你先本地跑起来，然后弄个新的分支，逐步将提交 pick 到新分支。注意不要乱改配置，也不要改无关的文件代码。
https://github.com/tisfeng/easydict#%E5%BC%80%E5%8F%91%E8%80%85%E6%9E%84%E5%BB%BA
There was a problem hiding this comment.
删除 Xcode 中空的 Niutrans，参照下面选项，将外部的 Niutrans 目录拖到 Xcode 的 Service 下面
|
@Kyle-Ye 有空帮忙看一下这个 PR，他刚接触 objc，不知道怎么更改了一些配置文件。。 |
There was a problem hiding this comment.
最小改动原则，先回滚下相关的改动吧，只保留 Easydic.xcodeproj 和 Easydic 文件夹以及 README 的改动
@tisfeng 后续合入的时候记得squash merge下，或者 @BigGuang97 你先本地重新整理 commit 后把这个分支 push --force 下
…e, systemPreferredLanguages[1] crash
|
@BigGuang97 你好，请尽快解决前面提到的问题，如果需要帮助，我这边也可以提供。 |
|
已在 #239 合并。 |
#222