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

Improve categorization on wall of apps page

    Details

    • Type: Improvement
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: None
    • Labels:
      None
    • Story Points:
      3
    • Sprint:
      Fall 8 : 25 Nov to 6 Dec, Fall 9 : 9 Dec to 20 Dec, Spring 1 : 25 Dec to 17 Jan

      Description

      On the 'Wall of Apps" page, categories display all the versions/releases of an app matching that category, resulting in repeats. Different apps belonging to the same category are not placed in the same category row; rather, a multiple rows of a that category are created, with each app in its own row.

      Objective:

      • Ensure that a given app appears no more than once in each category/row.
      • Ensure that categories aren't duplicated and have a single dedicated row
      • Correct total app count to represent the sum of all unique apps with categories

        Attachments

          Issue Links

            Activity

            pbadzuh Philip Badzuh (Inactive) created issue -
            pbadzuh Philip Badzuh (Inactive) made changes -
            Field Original Value New Value
            Link This issue relates to REL-33 [ REL-33 ]
            pbadzuh Philip Badzuh (Inactive) made changes -
            Link This issue relates to IGBF-2036 [ IGBF-2036 ]
            pbadzuh Philip Badzuh (Inactive) made changes -
            Epic Link IGBF-1388 [ 17463 ]
            pbadzuh Philip Badzuh (Inactive) made changes -
            Status To-Do [ 10305 ] In Progress [ 3 ]
            ann.loraine Ann Loraine made changes -
            Sprint Fall 8 : 25 Nov to 6 Dec [ 79 ] Fall 8 : 25 Nov to 6 Dec, Fall 9 : 9 Dec to 20 Dec [ 79, 80 ]
            ann.loraine Ann Loraine made changes -
            Rank Ranked higher
            pbadzuh Philip Badzuh (Inactive) made changes -
            Description On the 'Wall of Apps" page, categories display all the versions of an app matching that category, including repeats of the same app if it has multiple versions/releases.

            - Ensure that apps appear no more than once in each category.
            - Ensure that categories aren't duplicated
            On the 'Wall of Apps" page, categories display all the versions/releases of an app matching that category, resulting in repeats. Different apps belonging to the same category are not placed in the same category row; rather, a multiple rows of a that category are created, with each app in its own row.

            Objective:
            - Ensure that a given app appears no more than once in each category/row.
            - Ensure that categories aren't duplicated and have a single dedicated row
            pbadzuh Philip Badzuh (Inactive) made changes -
            Story Points 1 2
            Hide
            pbadzuh Philip Badzuh (Inactive) added a comment - - edited

            I have added the objective changes here. Please review and suggest any changes:
            https://bitbucket.org/pbadzuh/appstore_pbdev/branch/IGBF-2190#diff

            Show
            pbadzuh Philip Badzuh (Inactive) added a comment - - edited I have added the objective changes here. Please review and suggest any changes: https://bitbucket.org/pbadzuh/appstore_pbdev/branch/IGBF-2190#diff
            Hide
            pbadzuh Philip Badzuh (Inactive) added a comment - - edited

            Note that uncategorized apps are not currently displayed on the 'wall of apps' page. To add this feature, it may be convenient to create default 'Category' objects for new/uncategorized apps, as this data structure is defined in apps/models.py and used for the current construction of 'wall of apps'.

            The current 'other' category refers to apps that are not in the top x categories, where x can be set and is currently 20. Uncategorized apps could be added to this category in the future, as described in the first paragraph.

            Show
            pbadzuh Philip Badzuh (Inactive) added a comment - - edited Note that uncategorized apps are not currently displayed on the 'wall of apps' page. To add this feature, it may be convenient to create default 'Category' objects for new/uncategorized apps, as this data structure is defined in apps/models.py and used for the current construction of 'wall of apps'. The current 'other' category refers to apps that are not in the top x categories, where x can be set and is currently 20. Uncategorized apps could be added to this category in the future, as described in the first paragraph.
            pbadzuh Philip Badzuh (Inactive) made changes -
            Attachment old_categorization.png [ 14557 ]
            Attachment new_categorization.png [ 14558 ]
            pbadzuh Philip Badzuh (Inactive) made changes -
            Description On the 'Wall of Apps" page, categories display all the versions/releases of an app matching that category, resulting in repeats. Different apps belonging to the same category are not placed in the same category row; rather, a multiple rows of a that category are created, with each app in its own row.

            Objective:
            - Ensure that a given app appears no more than once in each category/row.
            - Ensure that categories aren't duplicated and have a single dedicated row
            On the 'Wall of Apps" page, categories display all the versions/releases of an app matching that category, resulting in repeats. Different apps belonging to the same category are not placed in the same category row; rather, a multiple rows of a that category are created, with each app in its own row.

            Objective:
            - Ensure that a given app appears no more than once in each category/row.
            - Ensure that categories aren't duplicated and have a single dedicated row
            - Correct
            pbadzuh Philip Badzuh (Inactive) made changes -
            Description On the 'Wall of Apps" page, categories display all the versions/releases of an app matching that category, resulting in repeats. Different apps belonging to the same category are not placed in the same category row; rather, a multiple rows of a that category are created, with each app in its own row.

            Objective:
            - Ensure that a given app appears no more than once in each category/row.
            - Ensure that categories aren't duplicated and have a single dedicated row
            - Correct
            On the 'Wall of Apps" page, categories display all the versions/releases of an app matching that category, resulting in repeats. Different apps belonging to the same category are not placed in the same category row; rather, a multiple rows of a that category are created, with each app in its own row.

            Objective:
            - Ensure that a given app appears no more than once in each category/row.
            - Ensure that categories aren't duplicated and have a single dedicated row
            - Correct total app count to represent the sum of all unique apps with categories
            Hide
            pbadzuh Philip Badzuh (Inactive) added a comment -

            Note 2: Category rows are ordered by the number of apps in each category, however, if a category comes to exceed another in its app number, row ordering will not be changed, despite hard refresh of the webpage, until the django server is restarted.

            Show
            pbadzuh Philip Badzuh (Inactive) added a comment - Note 2: Category rows are ordered by the number of apps in each category, however, if a category comes to exceed another in its app number, row ordering will not be changed, despite hard refresh of the webpage, until the django server is restarted.
            pbadzuh Philip Badzuh (Inactive) made changes -
            Attachment new_categorization.png [ 14558 ]
            pbadzuh Philip Badzuh (Inactive) made changes -
            Attachment new_categorization.png [ 14559 ]
            pbadzuh Philip Badzuh (Inactive) made changes -
            Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
            pbadzuh Philip Badzuh (Inactive) made changes -
            Assignee Philip Badzuh [ pbadzuh ]
            ann.loraine Ann Loraine made changes -
            Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
            ann.loraine Ann Loraine made changes -
            Assignee Ann Loraine [ aloraine ]
            Hide
            ann.loraine Ann Loraine added a comment -

            Philip Badzuh:

            Please check that an App will still be visible on the Wall of Apps even if it does not yet have a logo.

            Show
            ann.loraine Ann Loraine added a comment - Philip Badzuh : Please check that an App will still be visible on the Wall of Apps even if it does not yet have a logo.
            ann.loraine Ann Loraine made changes -
            Status First Level Review in Progress [ 10301 ] To-Do [ 10305 ]
            ann.loraine Ann Loraine made changes -
            Assignee Ann Loraine [ aloraine ] Philip Badzuh [ pbadzuh ]
            pbadzuh Philip Badzuh (Inactive) made changes -
            Attachment new_categorization.png [ 14559 ]
            pbadzuh Philip Badzuh (Inactive) made changes -
            Attachment new_categorization.png [ 14560 ]
            Hide
            pbadzuh Philip Badzuh (Inactive) added a comment -

            Yes, that is the case. I have updated the screenshot, so you can see it there.

            Show
            pbadzuh Philip Badzuh (Inactive) added a comment - Yes, that is the case. I have updated the screenshot, so you can see it there.
            pbadzuh Philip Badzuh (Inactive) made changes -
            Status To-Do [ 10305 ] In Progress [ 3 ]
            pbadzuh Philip Badzuh (Inactive) made changes -
            Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
            pbadzuh Philip Badzuh (Inactive) made changes -
            Assignee Philip Badzuh [ pbadzuh ]
            Hide
            ann.loraine Ann Loraine added a comment -

            OK - looks good! Please proceed with PR.

            Show
            ann.loraine Ann Loraine added a comment - OK - looks good! Please proceed with PR.
            ann.loraine Ann Loraine made changes -
            Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
            ann.loraine Ann Loraine made changes -
            Status First Level Review in Progress [ 10301 ] Ready for Pull Request [ 10304 ]
            ann.loraine Ann Loraine made changes -
            Assignee Philip Badzuh [ pbadzuh ]
            Show
            pbadzuh Philip Badzuh (Inactive) added a comment - Please see PR here: https://bitbucket.org/lorainelab/appstore/pull-requests/197/igbf-2190-improve-categorization-on/diff
            ann.loraine Ann Loraine made changes -
            Status Ready for Pull Request [ 10304 ] Pull Request Submitted [ 10101 ]
            ann.loraine Ann Loraine made changes -
            Status Pull Request Submitted [ 10101 ] Reviewing Pull Request [ 10303 ]
            ann.loraine Ann Loraine made changes -
            Status Reviewing Pull Request [ 10303 ] Merged Needs Testing [ 10002 ]
            ann.loraine Ann Loraine made changes -
            Assignee Philip Badzuh [ pbadzuh ]
            Hide
            ann.loraine Ann Loraine added a comment -

            Merged and deployed on https://dev-appstore-2.bioviz.org.

            Show
            ann.loraine Ann Loraine added a comment - Merged and deployed on https://dev-appstore-2.bioviz.org .
            shamika Shamika Gajanan Kulkarni (Inactive) made changes -
            Attachment Categories1.JPG [ 14561 ]
            Attachment WallofApps1.JPG [ 14562 ]
            shamika Shamika Gajanan Kulkarni (Inactive) made changes -
            Attachment WallofApps1.JPG [ 14562 ]
            shamika Shamika Gajanan Kulkarni (Inactive) made changes -
            Attachment Categories1.JPG [ 14561 ]
            shamika Shamika Gajanan Kulkarni (Inactive) made changes -
            Attachment S - Categories1.JPG [ 14563 ]
            Attachment S - WallofApps1.JPG [ 14564 ]
            Hide
            shamika Shamika Gajanan Kulkarni (Inactive) added a comment -

            Tested the ticket. I tried to add categories and checked the Wall of Apps. At that time, it worked perfectly fine. But when I tried to update the app and remove some categories, they were still visible on Wall of Apps page, with count as (0), and no apps displayed for those categories. Also, on the left menu on home page, one category called 'Test App' was not displayed even after refreshing the page, when an app existed for the category.
            Philip Badzuh could you please have a look into this?

            Another observation is that when u remove a category previously added for an app, it still exists on the left menu of homepage. I am not sure whether it is supposed to be like this so thought of noting it down here.

            Kindly refer the screenshots attached.

            Show
            shamika Shamika Gajanan Kulkarni (Inactive) added a comment - Tested the ticket. I tried to add categories and checked the Wall of Apps. At that time, it worked perfectly fine. But when I tried to update the app and remove some categories, they were still visible on Wall of Apps page, with count as (0), and no apps displayed for those categories. Also, on the left menu on home page, one category called 'Test App' was not displayed even after refreshing the page, when an app existed for the category. Philip Badzuh could you please have a look into this? Another observation is that when u remove a category previously added for an app, it still exists on the left menu of homepage. I am not sure whether it is supposed to be like this so thought of noting it down here. Kindly refer the screenshots attached.
            shamika Shamika Gajanan Kulkarni (Inactive) made changes -
            Status Merged Needs Testing [ 10002 ] Post-merge Testing In Progress [ 10003 ]
            shamika Shamika Gajanan Kulkarni (Inactive) made changes -
            Assignee Shamika Gajanan Kulkarni [ shamika ]
            shamika Shamika Gajanan Kulkarni (Inactive) made changes -
            Assignee Shamika Gajanan Kulkarni [ shamika ] Philip Badzuh [ pbadzuh ]
            shamika Shamika Gajanan Kulkarni (Inactive) made changes -
            Status Post-merge Testing In Progress [ 10003 ] To-Do [ 10305 ]
            pbadzuh Philip Badzuh (Inactive) made changes -
            Status To-Do [ 10305 ] In Progress [ 3 ]
            Hide
            pbadzuh Philip Badzuh (Inactive) added a comment -
            Show
            pbadzuh Philip Badzuh (Inactive) added a comment - I have fixed the issue I found here as well as those reported by Shamika Gajanan Kulkarni in the commit below: https://bitbucket.org/pbadzuh/appstore_pbdev/commits/46dd28f4362ea539fced1fb19bb66998914d5a10?at=IGBF-2190
            pbadzuh Philip Badzuh (Inactive) made changes -
            Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
            pbadzuh Philip Badzuh (Inactive) made changes -
            Story Points 2 3
            Assignee Philip Badzuh [ pbadzuh ]
            ann.loraine Ann Loraine made changes -
            Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
            ann.loraine Ann Loraine made changes -
            Status First Level Review in Progress [ 10301 ] To-Do [ 10305 ]
            Hide
            ann.loraine Ann Loraine added a comment -

            Please see question regarding commit: "IGBF 2190 Delete empty categories"

            (Question is a note/comment attached to the commit and referencing the bit in mk_app_page where it seems like tags are getting removed from the database.)

            Show
            ann.loraine Ann Loraine added a comment - Please see question regarding commit: "IGBF 2190 Delete empty categories" (Question is a note/comment attached to the commit and referencing the bit in mk_app_page where it seems like tags are getting removed from the database.)
            ann.loraine Ann Loraine made changes -
            Assignee Philip Badzuh [ pbadzuh ]
            pbadzuh Philip Badzuh (Inactive) made changes -
            Status To-Do [ 10305 ] In Progress [ 3 ]
            Hide
            pbadzuh Philip Badzuh (Inactive) added a comment - - edited

            I have reverted deleting empty app categories and have made it so they aren't displayed on the 'wall of apps' and home pages. They are still however in the list of suggested apps that is given when a user tries to add a category tag to an app.

            The changes can be seen in the branch below. Kindly review.
            https://bitbucket.org/pbadzuh/appstore_pbdev/branch/IGBF-2190#diff

            Show
            pbadzuh Philip Badzuh (Inactive) added a comment - - edited I have reverted deleting empty app categories and have made it so they aren't displayed on the 'wall of apps' and home pages. They are still however in the list of suggested apps that is given when a user tries to add a category tag to an app. The changes can be seen in the branch below. Kindly review. https://bitbucket.org/pbadzuh/appstore_pbdev/branch/IGBF-2190#diff
            pbadzuh Philip Badzuh (Inactive) made changes -
            Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
            pbadzuh Philip Badzuh (Inactive) made changes -
            Assignee Philip Badzuh [ pbadzuh ]
            Show
            pbadzuh Philip Badzuh (Inactive) added a comment - PR: https://bitbucket.org/lorainelab/appstore/pull-requests/202/igbf-2190-remove-empty-categories-from/diff
            pbadzuh Philip Badzuh (Inactive) made changes -
            Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
            pbadzuh Philip Badzuh (Inactive) made changes -
            Status First Level Review in Progress [ 10301 ] Ready for Pull Request [ 10304 ]
            pbadzuh Philip Badzuh (Inactive) made changes -
            Status Ready for Pull Request [ 10304 ] Pull Request Submitted [ 10101 ]
            ann.loraine Ann Loraine made changes -
            Status Pull Request Submitted [ 10101 ] Reviewing Pull Request [ 10303 ]
            ann.loraine Ann Loraine made changes -
            Status Reviewing Pull Request [ 10303 ] Merged Needs Testing [ 10002 ]
            ann.loraine Ann Loraine made changes -
            Sprint Fall 8 : 25 Nov to 6 Dec, Fall 9 : 9 Dec to 20 Dec [ 79, 80 ] Fall 8 : 25 Nov to 6 Dec, Fall 9 : 9 Dec to 20 Dec, Spring 1 : 6 Jan to 17 Jan [ 79, 80, 84 ]
            ann.loraine Ann Loraine made changes -
            Rank Ranked higher
            ann.loraine Ann Loraine made changes -
            Status Merged Needs Testing [ 10002 ] Post-merge Testing In Progress [ 10003 ]
            ann.loraine Ann Loraine made changes -
            Resolution Done [ 10000 ]
            Status Post-merge Testing In Progress [ 10003 ] Closed [ 6 ]
            ann.loraine Ann Loraine made changes -
            Assignee Philip Badzuh [ pbadzuh ]

              People

              • Assignee:
                pbadzuh Philip Badzuh (Inactive)
                Reporter:
                pbadzuh Philip Badzuh (Inactive)
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: