Skip to content

Comments

[add] import script for Organization data#4

Merged
TechQuery merged 47 commits intomasterfrom
feat/import-data-script
Aug 14, 2025
Merged

[add] import script for Organization data#4
TechQuery merged 47 commits intomasterfrom
feat/import-data-script

Conversation

@luojiyin1987
Copy link
Contributor

@luojiyin1987 luojiyin1987 commented Aug 11, 2025

PR-4 PR-4 PR-4 Powered by Pull Request Badge

@luojiyin1987 luojiyin1987 requested a review from TechQuery August 11, 2025 08:26
@luojiyin1987 luojiyin1987 requested a review from TechQuery August 12, 2025 02:42
Copy link
Member

@TechQuery TechQuery left a comment

Choose a reason for hiding this comment

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

类似的问题先改一下,并把主脚本文件代码量降到 400 行以内,我再审。

@luojiyin1987 luojiyin1987 requested a review from TechQuery August 12, 2025 09:50
@luojiyin1987 luojiyin1987 requested a review from TechQuery August 13, 2025 04:40
luojiyin1987 and others added 3 commits August 13, 2025 16:33
[optimize] simplify Import Type with Project built-in type package
@TechQuery TechQuery added the feature New feature or request label Aug 13, 2025
@github-project-automation github-project-automation bot moved this from Ready to In progress in 官网项目群 Aug 14, 2025
@luojiyin1987 luojiyin1987 requested a review from TechQuery August 14, 2025 15:52
@github-project-automation github-project-automation bot moved this from In progress to In review in 官网项目群 Aug 14, 2025
@TechQuery TechQuery changed the title Feat/import data script [add] import script for Organization data Aug 14, 2025
@TechQuery TechQuery merged commit b861cf9 into master Aug 14, 2025
2 checks passed
@TechQuery TechQuery deleted the feat/import-data-script branch August 14, 2025 16:12
@github-project-automation github-project-automation bot moved this from In review to Done in 官网项目群 Aug 14, 2025
Comment on lines +36 to +47
async createUser(data: UserData) {
const { body } = await this.client.post<UserData>('/api/users', data);

return body;
}

async findUserByEmail(email: string) {
const { body } = await this.client.get<UserData[]>(
`/api/users?filters[email][$eq]=${encodeURIComponent(email)}`,
);
return body[0];
}
Copy link
Member

Choose a reason for hiding this comment

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

@luojiyin1987 这两个方法完全没用上……

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

Labels

feature New feature or request

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants