Uploaded image for project: 'Guacamole'
  1. Guacamole
  2. GUACAMOLE-1550

Clarify TOTP reset/status logic

VotersWatch issueWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Trivial
    • Resolution: Done
    • 1.4.0
    • 1.6.0
    • guacamole-auth-totp
    • None

    Description

      The current TOTP reset logic as of GUACAMOLE-770 works well, but feels somewhat clunky in its use of a checkbox to request that TOTP status be cleared. Checkboxes typically represent state, but the TOTP form leverages checkboxes to represent both an action and current state:

      Rather than conceptualize things as an action ("reset") and represent that action with a checkbox, it would be better to consistently use checkboxes to represent state. For example:

      Attachments

        1. totp-reset-form.png
          7 kB
          Mike Jumper
        2. totp-status-form.png
          8 kB
          Mike Jumper

        Issue Links

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            mjumper Mike Jumper
            mjumper Mike Jumper
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment