Skip to content

Remove Perps app link from the Apps section - #509

Merged
ch4r10t33r merged 1 commit into
stagingfrom
chore/remove-perps-from-apps
Aug 18, 2026
Merged

Remove Perps app link from the Apps section#509
ch4r10t33r merged 1 commit into
stagingfrom
chore/remove-perps-from-apps

Conversation

@ch4r10t33r

Copy link
Copy Markdown
Contributor

Summary

Removes the Perps entry from the Apps section. The Perps app was never returned by the apps API — AllowedAppsProvider appended it client-side (and also used it as the fallback list when the API call fails). This PR drops that local injection; the provider now shows exactly what the API returns.

Per scope, no perps code is removedsrc/apps/perps/ and its /perps route remain intact, so the app is still reachable by direct URL.

Changes

  • src/providers/AllowedAppsProvider.tsx: stop appending the local perps-local entry on success and on API failure (failure now yields an empty list)
  • src/providers/__tests__/AllowedAppsProvider.test.tsx: drop the perps expectation

Test

vitest run src/providers/__tests__/AllowedAppsProvider.test.tsx — 2/2 passing.

Note for reviewers

The home feed still renders PerpsTile (src/apps/pillarx-app/index.tsx), a Hyperliquid promo tile linking to /perps. Left in place since this PR is scoped to the Apps section — shout if that should go too.

@coderabbitai

coderabbitai Bot commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Important

Review available on request

  • 🔍 Trigger review

Reviews should be triggered manually for repositories with fewer than 10 stars. Select Trigger review above or comment @coderabbitai review to review the latest changes. For a full review, comment @coderabbitai full review.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: bb0f1f60-5425-4f64-b31e-be83cca03c10


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@ch4r10t33r
ch4r10t33r merged commit 72c6d2a into staging Aug 18, 2026
7 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants