Skip to content

Commit

Permalink
Merge pull request #225 from VEuPathDB/161-only-apps-with-plugins
Browse files Browse the repository at this point in the history
Remove hardcoded apps filtering in SAM and only display apps with plugins
  • Loading branch information
bobular authored May 22, 2023
2 parents 64d999f + 778fdd4 commit 17795e9
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -54,8 +54,9 @@ export function StartPage(props: Props) {
}}
>
{/* orderBy renders available apps ahead of those in development */}
{orderBy(apps, [(app) => (plugins[app.name] ? 1 : 0)], ['desc']).map(
(app) => (
{apps
.filter((app) => plugins[app.name] != null)
.map((app) => (
<div
style={{
display: 'grid',
Expand Down Expand Up @@ -224,8 +225,7 @@ export function StartPage(props: Props) {
})}
</div>
</div>
)
)}
))}
</div>
</div>
)
Expand Down
2 changes: 1 addition & 1 deletion packages/libs/eda/src/lib/map/analysis/MapAnalysis.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -332,7 +332,7 @@ function MapAnalysisImpl(props: Props & CompleteAppState) {
const appsPromiseState = usePromise(
useCallback(async () => {
const { apps } = await dataClient.getApps();
return apps.filter((app) => app.name.startsWith('standalone-map-')); // TO DO: remove this temporary hack
return apps; // return all apps; new viz picker will only show those with client plugins defined
}, [dataClient])
);

Expand Down

0 comments on commit 17795e9

Please sign in to comment.