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

NIT 1452 ldap cli fix error handling #64

Closed
wants to merge 15 commits into from

Conversation

georgepstaylor
Copy link
Member

  • 🐛 Fix incorrect module path for update user roles (🐛 Fix incorrect module path for update user roles #51)

  • 🚑 Update User Roles: Use sync search and add summary (🚑 Update User Roles: Use sync search and add summary #52)

    • 🚑 Use sync search and add summary for searches and actions

    • Formatted code with black --line-length 120


    Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>

  • Better summary for update user roles job (Better summary for update user roles job #53)

    • 🚑 Use sync search and add summary for searches and actions

    • Formatted code with black --line-length 120

    • better debugging


    Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>

  • Update user.py

  • 🚑 Correct logic for matched user sets + role filtering (🚑 Correct logic for matched user sets + role filtering #55)

    • 🚑 correct logic for matched user sets + role filtering

    • Formatted code with black --line-length 120

    • Update role filter logic in user.py


    Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>

  • Fix update user role cmd (Fix update user role cmd #56)

    • 🚑 correct logic for matched user sets + role filtering

    • Formatted code with black --line-length 120

    • Update role filter logic in user.py

    • Formatted code with black --line-length 120

    • aliasing

    • Update user.py


    Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>

  • Fix update user role cmd (Fix update user role cmd #57)

    • 🚑 correct logic for matched user sets + role filtering

    • Formatted code with black --line-length 120

    • Update role filter logic in user.py

    • Formatted code with black --line-length 120

    • aliasing

    • Update user.py

    • Update user.py

    • Formatted code with black --line-length 120


    Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>

  • ✨ Use paged search to support larger data sets (✨ Use paged search to support larger data sets #58)

    • 🚑 correct logic for matched user sets + role filtering

    • Formatted code with black --line-length 120

    • Update role filter logic in user.py

    • Formatted code with black --line-length 120

    • aliasing

    • Update user.py

    • Update user.py

    • Formatted code with black --line-length 120

    • implement paging because we love ldap ❤️

    • remove test case + add var for page size

    • Formatted code with black --line-length 120


    Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>

  • Bump setuptools from 71.1.0 to 72.0.0 (Bump setuptools from 71.1.0 to 72.0.0 #60)
    Bumps setuptools from 71.1.0 to 72.0.0.


    updated-dependencies:

    • dependency-name: setuptools
      dependency-type: direct:production
      update-type: version-update:semver-major
      ...

    Signed-off-by: dependabot[bot] [email protected]
    Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

  • 🐛 error handling for policy/role recreation

georgepstaylor and others added 15 commits July 23, 2024 16:19
* 🚑 Use sync search and add summary for searches and actions

* Formatted code with black --line-length 120

---------

Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
* 🚑 Use sync search and add summary for searches and actions

* Formatted code with black --line-length 120

* better debugging

---------

Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
* 🚑 correct logic for matched user sets + role filtering

* Formatted code with black --line-length 120

* Update role filter logic in user.py

---------

Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
* 🚑 correct logic for matched user sets + role filtering

* Formatted code with black --line-length 120

* Update role filter logic in user.py

* Formatted code with black --line-length 120

* aliasing

* Update user.py

---------

Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
* 🚑 correct logic for matched user sets + role filtering

* Formatted code with black --line-length 120

* Update role filter logic in user.py

* Formatted code with black --line-length 120

* aliasing

* Update user.py

* Update user.py

* Formatted code with black --line-length 120

---------

Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
* 🚑 correct logic for matched user sets + role filtering

* Formatted code with black --line-length 120

* Update role filter logic in user.py

* Formatted code with black --line-length 120

* aliasing

* Update user.py

* Update user.py

* Formatted code with black --line-length 120

* implement paging because we love ldap ❤️

* remove test case + add var for page size

* Formatted code with black --line-length 120

---------

Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
Bumps [setuptools](https://github.com/pypa/setuptools) from 71.1.0 to 72.0.0.
- [Release notes](https://github.com/pypa/setuptools/releases)
- [Changelog](https://github.com/pypa/setuptools/blob/main/NEWS.rst)
- [Commits](pypa/setuptools@v71.1.0...v72.0.0)

---
updated-dependencies:
- dependency-name: setuptools
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@georgepstaylor georgepstaylor requested a review from a team as a code owner August 8, 2024 15:21
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.

1 participant