Skip to content
New issue

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

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

Already on GitHub? # to your account

[fix-10386]LDAP user create with status enable version 2.0.6 #10508

Merged
merged 1 commit into from
Jun 20, 2022

Conversation

qingwli
Copy link
Member

@qingwli qingwli commented Jun 20, 2022

close #10386

part of #10425

dev pr is #10426, unit test is different as dev, so just cherry-pick the code of fix bug

@caishunfeng caishunfeng added this to the 2.0.6 milestone Jun 20, 2022
@caishunfeng caishunfeng requested a review from brave-lee June 20, 2022 06:02
@codecov-commenter
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (2.0.6-prepare@930c7d4). Click here to learn what that means.
The diff coverage is n/a.

@@               Coverage Diff                @@
##             2.0.6-prepare   #10508   +/-   ##
================================================
  Coverage                 ?   21.07%           
  Complexity               ?     1595           
================================================
  Files                    ?      536           
  Lines                    ?    22543           
  Branches                 ?     2674           
================================================
  Hits                     ?     4751           
  Misses                   ?    17306           
  Partials                 ?      486           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 930c7d4...e76096d. Read the comment docs.

@sonarqubecloud
Copy link

@caishunfeng caishunfeng merged commit 67566fd into apache:2.0.6-prepare Jun 20, 2022
@qingwli qingwli deleted the fix-ldap branch September 7, 2023 06:41
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants