Description
A clear and concise description of what the issue is.
There are no favorited (bookmarked) apps or engines that are discoverable. The FE makes the call passing in favorites=true but in reality this filter is never used by the BE and the same list of apps/engines is returned in both calls.
How to reproduce the issue
- Create an app or engine
- Make it discoverable
- Remove your access to the app or engine
- Go back to the proper catalog page - go into discoverable tab
- Favorites section / call should not be made
Metadata
Metadata
Assignees
Labels
No labels
Activity
[-]My Discoverable Projects/Engines - No Favorites Version[/-][+][BUG] My Discoverable Projects/Engines - No Favorites Version[/+]johbaxter commentedon Mar 4, 2025
@phkem
At the end of the day, there is no concept of Favorited Discoverable apps.
On the FE we make a Reactor Call to get Favorited Apps when switching to the discoverable tab. We shouldn't make that extra frontend call for MyDiscoverableProjects(onlyFavorites=[true]).
But to also answer your question,
We should not show the ability to bookmark the app either while looking at a discoverable app
ppatel9703 commentedon May 8, 2025
@rameshpaulraj Pls do a investigation prior to passing it off to either fe devs
kyang4 commentedon May 14, 2025
For reference, please see this related ticket: SEMOSS/Semoss#701
samarthKharote commentedon May 16, 2025
Recreated the issue, I found
In the SEMOSS UI, users can toggle between
My Apps
andDiscoverable
tabs. WithinMy Apps
, users have the ability to bookmark their favorite applications. The issue arises when:-A user bookmarks one app in
My Apps
, leaving others unbookmarked.-The user switches to the "Discoverable" tab.
-Upon returning to the
My Apps
tab, all apps appear bookmarked, not just the one initially selected.So far, debugged the backend files which are related to this bug.
While inspecting, I found, there might be a chance of wrong query creation for PROJECTPERMISSION table where logic is putting "favoritesOnly" as "TRUE" for every project/app.
Need to conduct tests with modifying the subquery for the "favoritesOnly" part.
samarthKharote commentedon May 23, 2025
Deeply investigated the bug from backend side. can conclude that the bug is from backend only.
Need to modify setProjectFavorite() part from
MyprojectsReactor.Class
>>SecurityProjectUtils.Class
filessamarthKharote commentedon May 23, 2025
I submitted my modification and raised a PR for it