Uploaded image for project: 'XNAT'
  1. XNAT
  2. XNAT-4306

Notifications Email Recipients aren't being updated

    Details

    • Sprint:
      Prebeta push
    • Rank:
      0|ii0v53:
    • Sprint:
      Prebeta push

      Description

      When I update any of the email addresses under Notifications > "Email Recipient(s) for...", the batch REST call returns a 200. However, when I refresh the page, it's still set onto the site admin. If I do a get to the notifications api in swagger, the email address still lists the site admin.

        Activity

        Hide
        moore.c@wustl.edu Charlie Moore added a comment -

        In a way, it's similar to XNAT-4260 (almost as if a fix was attempting, and it's being overzealous...?)

        Show
        moore.c@wustl.edu Charlie Moore added a comment - In a way, it's similar to XNAT-4260 (almost as if a fix was attempting, and it's being overzealous...?)
        Hide
        mfmckay@wustl.edu Mike McKay (Inactive) added a comment -

        Fixed by https://bitbucket.org/xnatdev/xdat/commits/d5c37341fde11169485951266c8a581c30626cbc and https://bitbucket.org/xnatdev/xnat-web/commits/edd4a00f65cb1ec799e5d13815646e4ef2c69cc4 . That error was causing the bulk save to fail. Notifications now work. Just make sure your admin email is valid, all the emails in the list of recipients are valid, and all of the emails match users (if notifications are restricted to users).

        Show
        mfmckay@wustl.edu Mike McKay (Inactive) added a comment - Fixed by https://bitbucket.org/xnatdev/xdat/commits/d5c37341fde11169485951266c8a581c30626cbc and https://bitbucket.org/xnatdev/xnat-web/commits/edd4a00f65cb1ec799e5d13815646e4ef2c69cc4 . That error was causing the bulk save to fail. Notifications now work. Just make sure your admin email is valid, all the emails in the list of recipients are valid, and all of the emails match users (if notifications are restricted to users).
        Hide
        mfmckay@wustl.edu Mike McKay (Inactive) added a comment -

        By the way, the notifications email recipients were working as intended a week ago. This bug was caused by another preference producing an error when doing a bulk save.

        Show
        mfmckay@wustl.edu Mike McKay (Inactive) added a comment - By the way, the notifications email recipients were working as intended a week ago. This bug was caused by another preference producing an error when doing a bulk save.

          People

          • Assignee:
            moore.c@wustl.edu Charlie Moore
            Reporter:
            moore.c@wustl.edu Charlie Moore
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Agile