Skip to content

Conversation

@MorrisJobke
Copy link
Contributor

  • introduced new route settings/users/{id}/mailAddress
  • kept old responses
  • better error messages
  • dropped lostpassword.php from settings/ajax
  • cleaned up the UserList.add() and hand in user object instead of
    each attribute as another parameter
  • check for change permission of mail address
  • proper response messages

fixes #12595
ref #12620

cc @blizzz @DeepDiver1975 @PVince81 @LukasReschke

re-incarnation of #12853

@DeepDiver1975
Copy link
Member

@owncloud-bot Retest this please.

@DeepDiver1975
Copy link
Member

tested - works 👍

@DeepDiver1975
Copy link
Member

as discussed - please verify lost password screen is not broken - THX

@MorrisJobke MorrisJobke force-pushed the user-mail-user-management branch from 9c9aec0 to 71bcdec Compare December 18, 2014 21:41
@MorrisJobke
Copy link
Contributor Author

Rebased

* introduced new route settings/users/{id}/mailAddress
* kept old responses
* better error messages
* dropped lostpassword.php from settings/ajax
* cleaned up the UserList.add() and hand in user object instead of
  each attribute as another parameter
* check for change permission of mail address
* proper response messages
@MorrisJobke MorrisJobke force-pushed the user-mail-user-management branch from 71bcdec to 5913af8 Compare December 18, 2014 21:43
@MorrisJobke
Copy link
Contributor Author

Tested and still works :)

@scrutinizer-notifier
Copy link

The inspection completed: 15 new issues, 9 updated code elements

@MorrisJobke
Copy link
Contributor Author

Jenkins PR #12952

@MorrisJobke
Copy link
Contributor Author

@owncloud-bot retest this please

@ghost
Copy link

ghost commented Dec 19, 2014

Refer to this link for build results (access rights to CI server needed):
https://ci.owncloud.org//job/pull-request-analyser-ng-simple/5108/

Build result: FAILURE

[...truncated 17 lines...]using GIT_SSH to set credentials using .gitcredentials to set credentials > git config --local credential.helper store --file=/tmp/git5847736303688410432.credentials # timeout=10 > git fetch --tags --progress https://github.com/owncloud/core.git +refs/pull/:refs/remotes/origin/pr/ > git config --local --remove-section credential # timeout=10 > git rev-parse origin/pr/12921/merge^{commit} # timeout=10 > git branch -a --contains 254db24ef848100a1181b9fc863b92eeb5f4eae2 # timeout=10 > git rev-parse remotes/origin/pr/12921/merge^{commit} # timeout=10Checking out Revision 254db24ef848100a1181b9fc863b92eeb5f4eae2 (origin/pr/12921/merge) > git config core.sparsecheckout # timeout=10 > git checkout -f 254db24ef848100a1181b9fc863b92eeb5f4eae2 > git rev-list 7261c782c54e5deb2c60309e58943168b53c57eb # timeout=10First time build. Skipping changelog. > git remote # timeout=10 > git submodule init # timeout=10 > git submodule sync # timeout=10 > git config --get remote.origin.url # timeout=10 > git submodule update --init --recursiveTriggering pull-request-analyser-ng-simple » SLAVEpull-request-analyser-ng-simple » SLAVE completed with result FAILUREStarted calculate disk usage of buildFinished Calculation of disk usage of build in 0 secondsStarted calculate disk usage of workspaceFinished Calculation of disk usage of workspace in 2 second
Test FAILed.

@DeepDiver1975
Copy link
Member

DeepDiver1975 added a commit that referenced this pull request Dec 19, 2014
Mail address of users is now changable in the user management
@DeepDiver1975 DeepDiver1975 merged commit 995f01f into master Dec 19, 2014
@DeepDiver1975 DeepDiver1975 deleted the user-mail-user-management branch December 19, 2014 08:51
@lock lock bot locked as resolved and limited conversation to collaborators Aug 14, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[User Management] Add / update email

4 participants