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.

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

feat: feishu用户导入支持指定部门列表open_department_id #186

Merged
merged 4 commits into from Apr 10, 2023

Conversation

ckyoung123421
Copy link
Contributor

在提出此拉取请求时,我确认了以下几点(请复选框):

  • 我已阅读并理解贡献者指南
  • 我已检查没有与此请求重复的拉取请求。
  • 我已经考虑过,并确认这份呈件对其他人很有价值。
  • 我接受此提交可能不会被使用,并根据维护人员的意愿关闭拉取请求。

填写PR内容:

@eryajf
Copy link
Owner

eryajf commented Apr 10, 2023

这次pr是上次pr的增强版本吗,是不是包含上次pr的能力,如果合并这个,是不是可以关闭另外一个

抱歉,因为最近比较忙,一直没有时间做验证,所以合并pr慢了,我会尽快抽出时间进行测试验证,并合并pr

@ckyoung123421
Copy link
Contributor Author

这次pr是上次pr的增强版本吗,是不是包含上次pr的能力,如果合并这个,是不是可以关闭另外一个

抱歉,因为最近比较忙,一直没有时间做验证,所以合并pr慢了,我会尽快抽出时间进行测试验证,并合并pr

是的,已经覆盖上次的功能了,可以直接输入多个部门的list,之前只支持单个部门作为根部门

@eryajf eryajf merged commit b7c2266 into eryajf:main Apr 10, 2023
2 of 3 checks passed
@ckyoung123421
Copy link
Contributor Author

@eryajf 老哥 你给我的贡献者名称写了个NULL

@eryajf
Copy link
Owner

eryajf commented May 8, 2023

@eryajf 老哥 你给我的贡献者名称写了个NULL

这个是由action自动生成的,看起来可能是有点问题,随后我来修复一下

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants