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

Registration options are a bit off

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.7
    • Component/s: Admin UI
    • Labels:
    • Environment:

      xnat-dev01

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

      Description

      1. If I check "Auto-enable User Registration?", it does correctly autoenable new users upon email verification, but when the user clicks the verification link it takes them back to the login page without any indication that they can now login.
      2. If you disable user comments on registration, it still shows the comment field when registering a new account.

        Activity

        Hide
        mfmckay@wustl.edu Mike McKay (Inactive) added a comment -

        #2 was fixed by https://bitbucket.org/xnatdev/xnat-web/commits/7ebe578b1937041de4a82a435667fc592e41abd5 and https://bitbucket.org/xnatdev/xdat/commits/e32f8230a0a038686ca6e9968dd14412839d5db1

        #1 was fixed by my earlier commit (https://bitbucket.org/xnatdev/xnat-web/commits/86d9d855c6a0b9f0f5c3bc5b20651e1fa148ba9d) where I deleted the javascript which was removing any non-logout message on the login page.

        Charlie Moore In the future, could you try as much as possible to create separate bugs for stuff like this? It's nice to be able to know what's fixed by closing a bug, and if a bug is a combo of two issues, then you have to convey the current state through comments or through editing the bug. But if whole sections of the AdminUI don't work at all, I think a single bug for that is cool.

        Show
        mfmckay@wustl.edu Mike McKay (Inactive) added a comment - #2 was fixed by https://bitbucket.org/xnatdev/xnat-web/commits/7ebe578b1937041de4a82a435667fc592e41abd5 and https://bitbucket.org/xnatdev/xdat/commits/e32f8230a0a038686ca6e9968dd14412839d5db1 #1 was fixed by my earlier commit ( https://bitbucket.org/xnatdev/xnat-web/commits/86d9d855c6a0b9f0f5c3bc5b20651e1fa148ba9d ) where I deleted the javascript which was removing any non-logout message on the login page. Charlie Moore In the future, could you try as much as possible to create separate bugs for stuff like this? It's nice to be able to know what's fixed by closing a bug, and if a bug is a combo of two issues, then you have to convey the current state through comments or through editing the bug. But if whole sections of the AdminUI don't work at all, I think a single bug for that is cool.
        Hide
        moore.c@wustl.edu Charlie Moore added a comment -

        Yeah, I can do that. It's a tradeoff with spamming JIRA/scrum board a bit more than already, but it's probably a good paradigm to have single option issues or large sections in a single issue, without going in between. It also makes it a bit easier to search for issues.

        Are all the bugs that are Resolved but still assigned to you ready to be looked at?

        Show
        moore.c@wustl.edu Charlie Moore added a comment - Yeah, I can do that. It's a tradeoff with spamming JIRA/scrum board a bit more than already, but it's probably a good paradigm to have single option issues or large sections in a single issue, without going in between. It also makes it a bit easier to search for issues. Are all the bugs that are Resolved but still assigned to you ready to be looked at?
        Hide
        moore.c@wustl.edu Charlie Moore added a comment -

        Yep, both of these are fixed on xnat-dev05. Thanks, Mike!

        Show
        moore.c@wustl.edu Charlie Moore added a comment - Yep, both of these are fixed on xnat-dev05. Thanks, Mike!

          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