Skip to content
Snippets Groups Projects

Error message for missing users and roles in linkahead_admin.py

Merged I. Nüske requested to merge f-linkahead_admin-error-message into dev

Summary

Added error messages when retrieving and updating an user or role with linkahead_admin. Addresses #200 (closed)

Focus

The code changes are in the second diff version, the other changes are exclusively indentation fixes for docstrings. Would it be useful to also add success messages for operations that do not print anything?

Check List for the Author

  • All automated tests pass
  • Reference related issues
  • Not necessary: Appropriate user and developer documentation for new code
  • Annotations in code

Check List for the Reviewer

  • I understand the intent of this MR
  • All automated tests pass
  • Appropriate user and developer documentation for new code
  • The intended behavior is reproducible
Edited by Florian Spreckelsen

Merge request reports

Pipeline #56665 passed with warnings

Pipeline: LinkAhead

#56666

    Pipeline passed with warnings for 78803284 on f-linkahead_admin-error-message

    Approval is optional

    Merged by Florian SpreckelsenFlorian Spreckelsen 5 months ago (Oct 23, 2024 12:47pm UTC)

    Loading

    Pipeline #56894 passed with warnings

    Pipeline: LinkAhead

    #56897

      Pipeline passed with warnings for eb79cfbd on dev

      Activity

      Filter activity
      • Approvals
      • Assignees & reviewers
      • Comments (from bots)
      • Comments (from users)
      • Commits & branches
      • Edits
      • Labels
      • Lock status
      • Mentions
      • Merge request status
      • Tracking
      Please register or sign in to reply
      Loading