Skip to content
Snippets Groups Projects

Fix the user update endpoint not working

Description

Resolves #134 (closed).

Changes

  • The password update function now actually updates
  • Errors during updates are properly logged now

Additions

None.

Test and Review

To be filled in by the reviewers

  • All of the methods are commented to expectation

  • The methods are tested to satisfaction

  • There are no unnecessary files present in the MR

  • The continuous integration has no problems with the MR

  • The MR is filled in as requested (including labels, milestones, and reviewers)

  • The documentation is up-to-date

  • All nullable parameters are marked as such

Edited by Erik Oudsen

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Luc Everse added priocritical typebug + 1 deleted label

    added priocritical typebug + 1 deleted label

  • Luc Everse changed the description

    changed the description

  • Erik Oudsen marked the checklist item All of the methods are commented to expectation as completed

    marked the checklist item All of the methods are commented to expectation as completed

  • Erik Oudsen marked the checklist item The methods are tested to satisfaction as completed

    marked the checklist item The methods are tested to satisfaction as completed

  • Erik Oudsen marked the checklist item There are no unnecessary files present in the MR as completed

    marked the checklist item There are no unnecessary files present in the MR as completed

  • Erik Oudsen marked the checklist item The continuous integration has no problems with the MR as completed

    marked the checklist item The continuous integration has no problems with the MR as completed

  • Erik Oudsen marked the checklist item The MR is filled in as requested (including labels, milestones, and reviewers) as completed

    marked the checklist item The MR is filled in as requested (including labels, milestones, and reviewers) as completed

  • Erik Oudsen marked the checklist item The documentation is up-to-date as completed

    marked the checklist item The documentation is up-to-date as completed

  • Erik Oudsen marked the checklist item All nullable parameters are marked as such as completed

    marked the checklist item All nullable parameters are marked as such as completed

  • Erik Oudsen approved this merge request

    approved this merge request

  • Ewoud Ruighaver approved this merge request

    approved this merge request

  • mentioned in commit 816f0c14

Please register or sign in to reply
Loading