Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Improve PostAPIHelper to stateless #19

Closed
ininmm opened this issue Dec 8, 2020 · 5 comments
Closed

Improve PostAPIHelper to stateless #19

ininmm opened this issue Dec 8, 2020 · 5 comments

Comments

@ininmm
Copy link
Collaborator

ininmm commented Dec 8, 2020

In GitLab by @Aidan79225 on Nov 19, 2020, 22:36
@ininmm
Copy link
Collaborator Author

ininmm commented Dec 8, 2020

In GitLab by @Aidan79225 on Nov 19, 2020, 22:37

marked this issue as related to #1

@ininmm
Copy link
Collaborator Author

ininmm commented Dec 8, 2020

In GitLab by @in_in_mm on Nov 25, 2020, 14:08

@Aidan79225 這邊你會 follow 我之前的做法下去重構嗎？

@ininmm
Copy link
Collaborator Author

ininmm commented Dec 8, 2020

In GitLab by @Aidan79225 on Nov 25, 2020, 14:33

可以 但我今晚會先上一個MR 切割一下ui和model所需的物件 然後下一個MR再改成你那作法

@ininmm
Copy link
Collaborator Author

ininmm commented Dec 8, 2020

In GitLab by @Aidan79225 on Nov 26, 2020, 00:38

後來一併改上了 在幫看一下 MERGE REQUEST

@ininmm
Copy link
Collaborator Author

ininmm commented Dec 8, 2020

In GitLab by @in_in_mm on Nov 26, 2020, 10:41

好的 其實我 telegram 有私聊你，不過剛剛看了一下好像還沒有改到 ViewModel 就還好

@ininmm ininmm closed this as completed Dec 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ininmm