Uploaded image for project: 'IGB'
  1. IGB
  2. IGBF-2166

Fix rendering of editor email address

    Details

    • Type: New Feature
    • Status: Closed (View Workflow)
    • Priority: Minor
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: None
    • Labels:
      None

      Description

      Noted in https://jira.transvar.org/browse/REL-40

      See screenshots in REL-40.

      3) Adding a new Editor is possible only if the new editor has previously logged into Appstore. (Not sure if this was the requirement, just commenting to bring it to notice)

      • Shamika Gajanan Kulkarni - this is expected functionality, but is a usability flaw. The "add editor" feature should indicate to the user that an editor must already have logged into App Store at least once

      4) If the email address of an editor is long, it goes out of the box and looks weird.

      • This should be fixed if possible.

      Shamika Gajanan Kulkarni

        Attachments

          Issue Links

            Activity

            aloraine Ann Loraine created issue -
            aloraine Ann Loraine made changes -
            Field Original Value New Value
            Epic Link IGBF-1388 [ 17463 ]
            aloraine Ann Loraine made changes -
            Link This issue relates to REL-40 [ REL-40 ]
            aloraine Ann Loraine made changes -
            Rank Ranked higher
            aloraine Ann Loraine made changes -
            Description Noted in https://jira.transvar.org/browse/REL-40

            3) Adding a new Editor is possible only if the new editor has previously logged into Appstore. (Not sure if this was the requirement, just commenting to bring it to notice)

            * [~shamika] - this is expected functionality, but is a usability flaw. The "add editor" feature should indicate to the user that an editor must already have logged into App Store at least once

            4) If the email address of an editor is long, it goes out of the box and looks weird.

            * This should be fixed if possible.

            [~shamika]
            Noted in https://jira.transvar.org/browse/REL-40

            See screenshots in REL-40.


            3) Adding a new Editor is possible only if the new editor has previously logged into Appstore. (Not sure if this was the requirement, just commenting to bring it to notice)

            * [~shamika] - this is expected functionality, but is a usability flaw. The "add editor" feature should indicate to the user that an editor must already have logged into App Store at least once

            4) If the email address of an editor is long, it goes out of the box and looks weird.

            * This should be fixed if possible.

            [~shamika]
            sameer Sameer Shanbhag (Inactive) made changes -
            Status To-Do [ 10305 ] In Progress [ 3 ]
            sameer Sameer Shanbhag (Inactive) made changes -
            Assignee Sameer Shanbhag [ sameer ]
            sameer Sameer Shanbhag (Inactive) made changes -
            Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
            Show
            sameer Sameer Shanbhag (Inactive) added a comment - The CSS Issue Fixed as a part of IGBF-2158 PR Submitted: https://bitbucket.org/lorainelab/appstore/pull-requests/177/igbf-2158-cosmetic-changes-to-app-page-and/diff
            sameer Sameer Shanbhag (Inactive) made changes -
            Assignee Sameer Shanbhag [ sameer ]
            Hide
            sameer Sameer Shanbhag (Inactive) added a comment -

            Just a Quick Note:
            Editors (People who can edit the Application Page) need to be registered on the Appstore before we can add him/her to the Editors List.
            Code Wise there is no constraint on the login.

            Show
            sameer Sameer Shanbhag (Inactive) added a comment - Just a Quick Note: Editors (People who can edit the Application Page) need to be registered on the Appstore before we can add him/her to the Editors List. Code Wise there is no constraint on the login.
            Hide
            aloraine Ann Loraine added a comment -

            Merged but not yet deployed to dev-appstore-2

            Show
            aloraine Ann Loraine added a comment - Merged but not yet deployed to dev-appstore-2
            aloraine Ann Loraine made changes -
            Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
            aloraine Ann Loraine made changes -
            Status First Level Review in Progress [ 10301 ] Ready for Pull Request [ 10304 ]
            aloraine Ann Loraine made changes -
            Status Ready for Pull Request [ 10304 ] Pull Request Submitted [ 10101 ]
            aloraine Ann Loraine made changes -
            Status Pull Request Submitted [ 10101 ] Reviewing Pull Request [ 10303 ]
            aloraine Ann Loraine made changes -
            Status Reviewing Pull Request [ 10303 ] Merged Needs Testing [ 10002 ]
            Hide
            aloraine Ann Loraine added a comment -

            Deployed to https://dev-appstore-2.bioviz.org and ready for testing.

            Show
            aloraine Ann Loraine added a comment - Deployed to https://dev-appstore-2.bioviz.org and ready for testing.
            shamika Shamika Gajanan Kulkarni (Inactive) made changes -
            Assignee Shamika Gajanan Kulkarni [ shamika ]
            shamika Shamika Gajanan Kulkarni (Inactive) made changes -
            Status Merged Needs Testing [ 10002 ] Post-merge Testing In Progress [ 10003 ]
            Hide
            shamika Shamika Gajanan Kulkarni (Inactive) added a comment -

            Tested the ticket on https://dev-appstore-2.bioviz.org. Now longer email ids are word wrapped due to changes made. Moving to closed.

            Show
            shamika Shamika Gajanan Kulkarni (Inactive) added a comment - Tested the ticket on https://dev-appstore-2.bioviz.org . Now longer email ids are word wrapped due to changes made. Moving to closed.
            shamika Shamika Gajanan Kulkarni (Inactive) made changes -
            Resolution Done [ 10000 ]
            Status Post-merge Testing In Progress [ 10003 ] Closed [ 6 ]
            shamika Shamika Gajanan Kulkarni (Inactive) made changes -
            Assignee Shamika Gajanan Kulkarni [ shamika ] Sameer Shanbhag [ sameer ]

              People

              • Assignee:
                sameer Sameer Shanbhag (Inactive)
                Reporter:
                aloraine Ann Loraine
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: