feat: add mobx-strapi #209
Open
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
TechQuery
requested changes
Feb 24, 2024
TechQuery
requested changes
Apr 17, 2024
models/Activity/Award.ts
Outdated
| @@ -22,8 +23,8 @@ export interface AwardAssignment | |||
| award: Award; | |||
| } | |||
|
|
|||
| export class AwardModel extends Stream<Award>(ListModel) { | |||
| client = sessionStore.client; | |||
| export class AwardModel extends Stream<Award>(StrapiListModel) { | |||
There was a problem hiding this comment.
Suggested change
| export class AwardModel extends Stream<Award>(StrapiListModel) { | |
| export class AwardModel extends StrapiListModel<Award> { |
Strapi 有能定位到具体页数的分页参数，不需要 Stream 流式加载。
There was a problem hiding this comment.
看不懂上面的两次修改
以前是附加流式加载特性，需要用 mixin（混入）的写法，也就是单继承语言实现多继承。
但 Strapi 不需要流式加载，就直接继承基类即可。
| @@ -22,8 +23,8 @@ export interface AwardAssignment | |||
| award: Award; | |||
| } | |||
|
|
|||
| export class AwardModel extends Stream<Award>(ListModel) { | |||
| client = sessionStore.client; | |||
There was a problem hiding this comment.
此处不要改动，已经配置好登录验证机制的 API 客户端就是放在这个 store 上的。
There was a problem hiding this comment.
现在遇到 类型“AwardModel”上不存在属性“baseURI 的报错
There was a problem hiding this comment.
此处不要改动，已经配置好登录验证机制的 API 客户端就是放在这个 store 上的。
关于 client 的问题是我昏头了，你改为 strapiClient 是对的，是我在迁移阶段还没把 client 移到 sessionStore 上。
There was a problem hiding this comment.
现在遇到 类型“AwardModel”上不存在属性“baseURI 的报错
非流式加载的列表必须声明对应数据结构 API 根路径：
参考：https://github.com/idea2app/MobX-RESTful/tree/main/wrapper/Strapi#modelarticlets
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.
Checklist（清单）:
Closes #204