Skip to content

Commit

Permalink
Fix some docs problems
Browse files Browse the repository at this point in the history
  • Loading branch information
lukasjuhrich committed Jul 11, 2024
1 parent 82c576a commit 8bb6f77
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 5 deletions.
1 change: 1 addition & 0 deletions doc/guides/docker.rst
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ Installing Docker and Docker Compose
------------------------------------
Prerequisites
*nothing*

You need to install

* `Docker-engine <https://docs.docker.com/engine/install/>`__ ``≥19.03.0``
Expand Down
5 changes: 3 additions & 2 deletions ldap_sync/sources/db.py
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ def _fetch_db_users(
) -> list[_UserProxyType]:
"""Fetch users to be synced, plus whether ``ldap_login_enabled`` is set.
If the `` ldap_login_enabled`` flag is not present,
If the ``ldap_login_enabled`` flag is not present,
we interpret this as ``should_be_blocked``.
:param session: The SQLAlchemy session to use
Expand Down Expand Up @@ -199,7 +199,7 @@ class _PropertyProxyType(NamedTuple):
def _fetch_db_properties(session: Session) -> list[_PropertyProxyType]:
"""Fetch the groups who should be synced.
Explicitly, this returns everything in :ref:`EXPORTED_PROPERTIES` together with
Explicitly, this returns everything in :data:`EXPORTED_PROPERTIES` together with
the current users having the respective property as members.
:param session: The SQLAlchemy session to use
Expand Down Expand Up @@ -246,6 +246,7 @@ def fetch_db_properties(
)


#: The properties of a user we export to LDAP.
EXPORTED_PROPERTIES = frozenset(
[
"network_access",
Expand Down
6 changes: 3 additions & 3 deletions pycroft/lib/user.py
Original file line number Diff line number Diff line change
Expand Up @@ -573,7 +573,7 @@ def edit_email(
:param email_forwarded: Boolean if emails should be forwarded
:param processor: User object of the processor, which issues the change
:param is_confirmed: If the email address is already confirmed
:return:Changed user object
:return: Changed user object
"""

if not can_target(user, processor):
Expand Down Expand Up @@ -623,8 +623,8 @@ def edit_birthdate(user: User, birthdate: date, processor: User) -> User:
:param user: User object to change
:param birthdate: New birthdate
:param processor:User object of the processor, which issues the change
:return:Changed user object
:param processor: User object of the processor, which issues the change
:return: Changed user object
"""

if not birthdate:
Expand Down

0 comments on commit 8bb6f77

Please sign in to comment.