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

            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.
            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.
            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.
            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.
            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.
            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
            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 .
            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.
            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
            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.)
            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
            Show
            pbadzuh Philip Badzuh (Inactive) added a comment - PR: https://bitbucket.org/lorainelab/appstore/pull-requests/202/igbf-2190-remove-empty-categories-from/diff

              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: